Mac: Remove a case where panels may become key when an autofill popup closes

chrome/browser/ui/cocoa/panels/panel_window_controller_cocoa.mm

Index: chrome/browser/ui/cocoa/panels/panel_window_controller_cocoa.mm
diff --git a/chrome/browser/ui/cocoa/panels/panel_window_controller_cocoa.mm b/chrome/browser/ui/cocoa/panels/panel_window_controller_cocoa.mm
index de949486869ba5a5b0b55d90ce4f7fccf7d6f06b..53cb9db2d04299e4df14058124c0b5e1da05ff50 100644
--- a/chrome/browser/ui/cocoa/panels/panel_window_controller_cocoa.mm
+++ b/chrome/browser/ui/cocoa/panels/panel_window_controller_cocoa.mm
@@ -95,7 +95,6 @@ const double kWidthOfMouseResizeArea = 15.0;
       [controller activationRequestedByPanel] ||
       [app isCyclingWindows] ||
       [self isKeyWindow] ||
-      [app previousKeyWindow] == self ||

[Scott Hess - ex-Googler, 2016/01/12 22:29:54]

AFAICT, the only reason |previousKeyWindows_| in
chrome_browser_application_mac.mm exists is to support this check.  Removing all
of that makes me a little nervous (what if there are interesting non-obvious
side effects?), but not removing all of that also makes me nervous.

In the bug comment #3, it sounds like the sequence should be:
1) key window is new tab, previous is unknown.
2) key window is chat window, previous is #1's tab.
3.1) key window is #1's tab again, previous is #2's chat window.
3.2) key window is autocomplete popup, previous is #1's tab (the login page).

AFAICT, the problem at #4 is that when the key window is the autocomplete popup,
the previous key window is the chat window, instead of #1's tab (well, or window
thereof).  That would imply that the code to update the previousKeyWindows_
vector isn't firing for some reason.

OR, the problem at #4 is that the autocomplete popup doesn't become key window,
but dismissing it does cause the search for the previous key window.  In that
case, the current key window is still the tab's window, and the previous key
window is the chat window, and it should be re-selecting the current key window
somehow.

[OR this needs more thought, again.  It's hard to get back into the flow of this
kind of change, since it's complicated visible code interacting with complicated
invisible (and undocumented) code.]

       [[app windows] count] == static_cast<NSUInteger>([controller numPanels]);
 }