From 0c1bc91e8832bca4fe6563ba77f9986be6164841 Mon Sep 17 00:00:00 2001 From: Shannon Booth Date: Tue, 7 Jan 2020 20:31:35 +1300 Subject: [PATCH] WSWindowServer: Remove uneccessary special casing From my testing I could not see this code doing anything. Listen the FIXME, and remove this. --- Servers/WindowServer/WSWindowManager.cpp | 6 ------ 1 file changed, 6 deletions(-) diff --git a/Servers/WindowServer/WSWindowManager.cpp b/Servers/WindowServer/WSWindowManager.cpp index 9c56c13756f..515abb4dcc2 100644 --- a/Servers/WindowServer/WSWindowManager.cpp +++ b/Servers/WindowServer/WSWindowManager.cpp @@ -708,12 +708,6 @@ void WSWindowManager::process_mouse_event(WSMouseEvent& event, WSWindow*& hovere deliver_mouse_event(*window, translated_event); } - // FIXME: Now that the menubar has a dedicated window, is this special-casing really necessary? - if (!active_window_is_modal() && menubar_rect().contains(event.position())) { - m_menu_manager.dispatch_event(event); - return; - } - if (!menu_manager().open_menu_stack().is_empty()) { auto* topmost_menu = menu_manager().open_menu_stack().last().ptr(); ASSERT(topmost_menu);