When navigation focuses a web contents, also activate its window.

content/browser/web_contents/web_contents_view_mac.mm

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #import <Carbon/Carbon.h>
 
 #import "content/browser/web_contents/web_contents_view_mac.h"
 
 #include <string>
 
@@ skipping 209 matching lines @@
   return rwhv ? rwhv->GetNativeView() : nil;
 }
 
 void WebContentsViewMac::Focus() {
   gfx::NativeView native_view = GetNativeViewForFocus();
   NSWindow* window = [native_view window];
   [window makeFirstResponder:native_view];
   if (![window isVisible])
     return;
   [window makeKeyAndOrderFront:nil];
+  [NSApp activateIgnoringOtherApps:YES];

[Avi (use Gerrit), 2017/03/02 23:26:18]

I could swear I've seen this change before.

Who else calls Focus? Are we ok with that now forcing the window to the top of
the pile no matter what?

[Sidney San Martín, 2017/03/08 05:02:29]

One of these?
https://codereview.chromium.org/2412983004
https://codereview.chromium.org/2420733002

I haven't found the One True Change to cover all of these cases at once, yet.

[Sidney San Martín, 2017/03/08 05:02:29]

I'd missed some callers… so, thanks! I found a more appropriate place.

 }
 
 void WebContentsViewMac::SetInitialFocus() {
   if (web_contents_->FocusLocationBarByDefault())
     web_contents_->SetFocusToLocationBar(false);
   else
     Focus();
 }
 
 void WebContentsViewMac::StoreFocus() {
@@ skipping 494 matching lines @@
 
 - (void)viewDidHide {
   [self updateWebContentsVisibility];
 }
 
 - (void)viewDidUnhide {
   [self updateWebContentsVisibility];
 }
 
 @end