Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1522)

Unified Diff: chrome/browser/extensions/extension_ui_util.h

Issue 1006273002: Show location for bookmark apps that have navigated across origin. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove change to web app frame applicability Created 5 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_ui_util.h
diff --git a/chrome/browser/extensions/extension_ui_util.h b/chrome/browser/extensions/extension_ui_util.h
index c49c2bc61376794983494ac2655bff5734a011bc..b7df26a6fcb24f52c9fe312a30cbdc1e45d6bb21 100644
--- a/chrome/browser/extensions/extension_ui_util.h
+++ b/chrome/browser/extensions/extension_ui_util.h
@@ -7,6 +7,7 @@
namespace content {
class BrowserContext;
+class WebContents;
}
namespace extensions {
@@ -46,6 +47,11 @@ bool ShouldDisplayInExtensionSettings(const Extension* extension,
bool ShouldNotBeVisible(const Extension* extension,
content::BrowserContext* context);
+// Returns true if the location bar should be shown for |web_contents| when
+// viewed in the context of the app represented by |extension|..
Lei Zhang 2015/03/17 07:19:56 nit: extra period at the end.
benwells 2015/03/17 10:28:13 Done.
+bool ShouldShowLocationBar(const Extension* extension,
+ content::WebContents* web_contents);
Lei Zhang 2015/03/17 07:19:56 this can probably be const as well.
benwells 2015/03/17 10:28:13 Done.
+
} // namespace ui_util
} // namespace extensions
« no previous file with comments | « no previous file | chrome/browser/extensions/extension_ui_util.cc » ('j') | chrome/browser/extensions/extension_ui_util.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698