Menus are no longer destroyed on Mac when focus is restored from another app.

R=avi

Bug: 1016011
Change-Id: I609c2958e52d837e1f43b0fd7497ba2013314086
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1872561Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Federico Paredes <fedpar@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#707974}

ui/views/controls/menu/menu_cocoa_watcher_mac.mm

@@ -33,7 +33,13 @@ MenuCocoaWatcherMac::MenuCocoaWatcherMac(base::OnceClosure callback)
                       object:nil
                        queue:nil
                   usingBlock:^(NSNotification* notification) {
-                    ExecuteCallback();
+                    // Only destroy menus if the browser is losing focus, not if
+                    // it's gaining focus. This is to ensure that we can invoke
+                    // a context menu while focused on another app, and still be
+                    // able to click on menu items without dismissing the menu.
+                    if (![[NSRunningApplication currentApplication] isActive]) {
+                      ExecuteCallback();
+                    }
                   }];
 }