Don't show location bar in bookmark apps if there has been no navigation

chrome/browser/extensions/extension_ui_util.cc

Index: chrome/browser/extensions/extension_ui_util.cc
diff --git a/chrome/browser/extensions/extension_ui_util.cc b/chrome/browser/extensions/extension_ui_util.cc
index fe1915594e13afcbe74ebaa52b2bb6329f011f5a..818b4acd7df3b51f01c2f23e46753fca13b62d2f 100644
--- a/chrome/browser/extensions/extension_ui_util.cc
+++ b/chrome/browser/extensions/extension_ui_util.cc
@@ -79,6 +79,10 @@ bool ShouldShowLocationBar(const Extension* extension,
   if (!extension->from_bookmark())
     return false;
 
+  // Don't show a location bar until a navigation has occurred.
+  if (web_contents->GetLastCommittedURL().is_empty())

[felt, 2015/03/23 20:41:56]

Can you test what happens here if you navigate to a page that shows a warning?

[benwells, 2015/03/24 05:14:59]

I've put a screenshot on the bug. Let me know if that is what you meant.

[felt, 2015/03/24 05:27:25]

Thanks, yes, that's what I meant. I wanted to check that the committed entry
wouldn't end up empty if you hit an interstitial on the first navigation, or
something weird like that.

+    return false;
+
   GURL launch_url = AppLaunchInfo::GetLaunchWebURL(extension);
   return !(IsSameOriginOrMoreSecure(launch_url,
                                     web_contents->GetVisibleURL()) &&