Show location for bookmark apps that have navigated across origin.

chrome/browser/extensions/extension_ui_util.cc

 // Copyright 2014 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/extensions/extension_ui_util.h"
 
 #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"
 
 namespace extensions {
 
 namespace {
 
 bool IsBlockedByPolicy(const Extension* app, content::BrowserContext* context) {
   Profile* profile = Profile::FromBrowserContext(context);
@@ skipping 32 matching lines @@
   return extension->ShouldDisplayInExtensionSettings() &&
       !util::IsEphemeralApp(extension->id(), context);
 }
 
 bool ShouldNotBeVisible(const Extension* extension,
                         content::BrowserContext* context) {
   return extension->ShouldNotBeVisible() ||
       util::IsEphemeralApp(extension->id(), context);
 }
 
+bool ShouldShowLocationBar(const Extension* extension,
+                           content::WebContents* web_contents) {
+  // Default to not showing the location bar.
+  if (!extension || !web_contents)
+    return false;
+
+  GURL launch_url = AppLaunchInfo::GetLaunchWebURL(extension);
+  return extension->from_bookmark() &&
+         web_contents->GetURL().GetOrigin() != launch_url.GetOrigin();

[Lei Zhang, 2015/03/17 07:19:56]

GetURL() has a big "DO NOT USE" comment.

[benwells, 2015/03/17 10:28:13]

So it does... Replaced with visible URL as that is what the users will see in
the location bar that is being shown to them.

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