Wireshark-commits: [Wireshark-commits] master c936112: Qt: Remove unnecessary code for context menu

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Mon, 26 Aug 2019 19:57:32 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=c9361120f0a7544d0fc95a24d0e51aaaaf09c89a
Submitter: "Roland Knall <rknall@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

c936112 by Roland Knall (rknall@xxxxxxxxx):

    Qt: Remove unnecessary code for context menus
    
    This code is no longer required, as context menus should
    be bound to local events.
    
    As for "Reloading LUA plugins" - the keyboard shortcut would
    be able to trigger while the menu is open, except Qt explicitely
    prevents that
    
    Change-Id: Id18a3a17d62bae1ff547a6ef96772e12a2535219
    Reviewed-on: https://code.wireshark.org/review/34371
    Petri-Dish: Roland Knall <rknall@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Roland Knall <rknall@xxxxxxxxx>
    

Actions performed:

    from  3870e6c   Qt: Make Apply/Prepare filter independent
     add  c936112   Qt: Remove unnecessary code for context menus


Summary of changes:
 ui/qt/main_window_slots.cpp |  1 -
 ui/qt/packet_list.cpp       | 54 +++++++++++++++++------------------
 ui/qt/packet_list.h         |  1 -
 ui/qt/proto_tree.cpp        | 69 +++++++++++++++++++++------------------------
 ui/qt/proto_tree.h          |  2 --
 5 files changed, 59 insertions(+), 68 deletions(-)