Show location for bookmark apps that have navigated across origin.

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 4cf4fa89775566d31d8bf15def03738d868f8219..3ae3ca2ce669a474a3dbcbc889c286aca122e41f 100644
--- a/chrome/browser/extensions/extension_ui_util.cc
+++ b/chrome/browser/extensions/extension_ui_util.cc
@@ -7,7 +7,9 @@
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/extensions/extension_constants.h"
+#include "chrome/common/extensions/manifest_handlers/app_launch_info.h"
 #include "chrome/common/pref_names.h"
+#include "content/public/browser/web_contents.h"
 #include "extensions/browser/extension_util.h"
 #include "extensions/common/constants.h"
 #include "extensions/common/extension.h"
@@ -60,5 +62,16 @@ bool ShouldNotBeVisible(const Extension* extension,
       util::IsEphemeralApp(extension->id(), context);
 }
 
+bool ShouldShowLocationBar(const Extension* extension,
+                           const content::WebContents* web_contents) {
+  // Default to not showing the location bar.
+  if (!extension || !web_contents)
+    return false;

[felt, 2015/03/17 14:52:54]

Is it possible for either |extension| or |web_contents| to ever be null here in
normal browsing operations? If they should always have a value, would DCHECK be
better here?

[benwells, 2015/03/18 08:25:37]

The extension is null when the window is for the devtools. The web_contents
returned by tab_strip_model_->GetActiveWebContents looks like it can be null
(based on the tab_strip_model.h comments and other usage). To be honest I'm not
sure when it can be null.

+
+  GURL launch_url = AppLaunchInfo::GetLaunchWebURL(extension);
+  return extension->from_bookmark() &&
+         web_contents->GetVisibleURL().GetOrigin() != launch_url.GetOrigin();

[felt, 2015/03/17 14:52:54]

I think you might want to return true if either GetVisibleURL or
GetLastCommittedURL are different from the original origin.

[benwells, 2015/03/18 08:25:37]

Done.

+}
+
 }  // namespace ui_util
 }  // namespace extensions