Wireshark · Wireshark-commits: [Wireshark-commits] master-2.0 8d8d1c9: Qt: Add a play button to the RTP Stream Analysis dialog.

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=8d8d1c974d43703650dd8a08f226c9c3a442d328
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-2.0
Repository: wireshark

Commits:

8d8d1c9 by Gerald Combs (gerald@xxxxxxxxxxxxx):

    Qt: Add a play button to the RTP Stream Analysis dialog.
    
    Rename the "Play Call" button to "Play Streams". Move the button
    creation code to a common routine. Use it to add a "Play Streams" button
    to the RTP Stream Analysis, similar to the GTK+ UI.
    
    Don't restrict RTP to IPv[46] as suggested by Michal. I don't have any
    RTP-over-Bluetooth captures so I can't test this directly.
    
    Change-Id: I4703cac1d5bf5b3ff0255d36da2c5164feb0547d
    Reviewed-on: https://code.wireshark.org/review/10888
    Petri-Dish: Gerald Combs <gerald@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Gerald Combs <gerald@xxxxxxxxxxxxx>
    (cherry picked from commit f274902be581b52db68c66f4864d85b7ea8fefe4)
    Reviewed-on: https://code.wireshark.org/review/10909
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  4c061df   Make sure we can compile without QtMultimedia.
    adds  8d8d1c9   Qt: Add a play button to the RTP Stream Analysis dialog.


Summary of changes:
 ui/qt/decode_as_dialog.ui     |    3 ---
 ui/qt/rtp_analysis_dialog.cpp |   47 +++++++++++++++++++++++++++++++++++------
 ui/qt/rtp_analysis_dialog.h   |    8 +++++++
 ui/qt/rtp_player_dialog.cpp   |   10 +++++++++
 ui/qt/rtp_player_dialog.h     |    9 ++++++++
 ui/qt/voip_calls_dialog.cpp   |    3 +--
 6 files changed, 69 insertions(+), 11 deletions(-)