InstantExtended: Committed NTP

content/browser/web_contents/web_contents_impl.cc

 // 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.
 
 #include "content/browser/web_contents/web_contents_impl.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
@@ skipping 1950 matching lines @@
   int id = StartDownload(host, url, image_size);
   favicon_download_map_[id] = callback;
   return id;
 }
 
 bool WebContentsImpl::FocusLocationBarByDefault() {
   WebUI* web_ui = GetWebUIForCurrentState();
   if (web_ui)
     return web_ui->ShouldFocusLocationBarByDefault();
   NavigationEntry* entry = controller_.GetActiveEntry();
-  return (entry && entry->GetURL() == GURL(chrome::kAboutBlankURL));
+  return (entry &&
+          (entry->GetVirtualURL() == GURL(chrome::kAboutBlankURL) ||
+           entry->GetVirtualURL() == GURL(chrome::kChromeUINewTabURL)));

[dhollowa, 2013/01/22 22:34:58]

Should this not test whether the InstantExteded feature is enabled?

[samarth, 2013/01/25 21:08:40]

Wouldn't that break the content/chrome separation? This is a no-op for 1993:
currently, checking the URL is not necessary because the NTP uses WebUI which
can set the focus bit on itself.

[dhollowa, 2013/01/29 02:37:53]

Ya, ok.

 }
 
 void WebContentsImpl::SetFocusToLocationBar(bool select_all) {
   if (delegate_)
     delegate_->SetFocusToLocationBar(select_all);
 }
 
 void WebContentsImpl::OnRegisterIntentService(
     const webkit_glue::WebIntentServiceData& data,
     bool user_gesture) {
@@ skipping 1453 matching lines @@
 
 BrowserPluginGuest* WebContentsImpl::GetBrowserPluginGuest() {
   return browser_plugin_guest_.get();
 }
 
 BrowserPluginEmbedder* WebContentsImpl::GetBrowserPluginEmbedder() {
   return browser_plugin_embedder_.get();
 }
 
 }  // namespace content