Merge pull request #27225 from brave/fix_split_view_menu_bubble_visib…

…ility Fixed split view menu bubble is not hidden after command runs
brave · Jan 15, 2025 · c0022fb · c0022fb
2 parents ee91613 + 0874be0
commit c0022fb
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/browser/ui/views/split_view/split_view_menu_bubble.cc b/browser/ui/views/split_view/split_view_menu_bubble.cc
@@ -82,6 +82,11 @@ SplitViewMenuBubble::SplitViewMenuBubble(Browser* browser, views::View* anchor)
     return base::BindRepeating(
         [](Browser* browser, int command_id, const ui::Event& event) {
           chrome::ExecuteCommand(browser, command_id);
+          // Some platforms steal this bubble's focus closing
+          // implicitly but some don't, so we need to close it explicitly.
+          if (g_bubble) {
+            g_bubble->GetWidget()->Close();
+          }
         },
         browser, command_id);
   };