InstantExtended: Committed NTP

content/browser/web_contents/web_contents_impl.cc

Index: content/browser/web_contents/web_contents_impl.cc
diff --git a/content/browser/web_contents/web_contents_impl.cc b/content/browser/web_contents/web_contents_impl.cc
index 545ebf339ac8f32938257a07ec784ff1945799d2..74e8d26be9084cb4547aa3f869370c1f2c31a7b2 100644
--- a/content/browser/web_contents/web_contents_impl.cc
+++ b/content/browser/web_contents/web_contents_impl.cc
@@ -1968,7 +1968,9 @@ bool WebContentsImpl::FocusLocationBarByDefault() {
   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) {