Don't focus the location bar for NTP navigations in non-selected tabs.

chrome/browser/ui/browser.cc

 // Copyright 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.
 
 #include "chrome/browser/ui/browser.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <string>
@@ skipping 1533 matching lines @@
   if (IsFastTabUnloadEnabled()) {
     *proceed_to_fire_unload =
         fast_unload_controller_->BeforeUnloadFired(web_contents, proceed);
   } else {
     *proceed_to_fire_unload =
         unload_controller_->BeforeUnloadFired(web_contents, proceed);
   }
 }
 
 bool Browser::ShouldFocusLocationBarByDefault(WebContents* source) {
+  // Focus if we're going to the NTP, but only if it's in the active tab.

[Peter Kasting, 2017/01/12 18:24:04]

Nit: This might be clearer as "Navigations in background tabs shouldn't change
the focus state of the omnibox, since it's associated with the foreground tab."

[Charlie Reis, 2017/01/12 19:26:54]

Done.

+  if (source != tab_strip_model_->GetActiveWebContents())
+    return false;

[Peter Kasting, 2017/01/12 18:24:04]

Does this mean that, when a user switches to a now-navigated-to-the-NTP
background tab, the location bar won't be focused, but it should be?  Of course
that wouldn't have worked before either, I'm just wondering if there's a
secondary bug here that a background NTP navigation should mark the saved state
for that omnibox as "focused".

[Charlie Reis, 2017/01/12 19:26:54]

Yes, you're right that there's a secondary bug, and it affects Stable and Canary
today.  I've filed https://crbug.com/680624 for it.

+
   const content::NavigationEntry* entry =
       source->GetController().GetActiveEntry();
   if (entry) {
     const GURL& url = entry->GetURL();
     const GURL& virtual_url = entry->GetVirtualURL();
     if ((url.SchemeIs(content::kChromeUIScheme) &&
          url.host_piece() == chrome::kChromeUINewTabHost) ||
         (virtual_url.SchemeIs(content::kChromeUIScheme) &&
          virtual_url.host_piece() == chrome::kChromeUINewTabHost)) {
       return true;
@@ skipping 1016 matching lines @@
     // new window later, thus we need to navigate the window now.
     if (contents) {
       contents->web_contents()->GetController().LoadURL(
           target_url, content::Referrer(), ui::PAGE_TRANSITION_LINK,
           std::string());  // No extra headers.
     }
   }
 
   return contents != NULL;
 }