| Index: chrome/browser/extensions/extension_web_ui.cc
|
| diff --git a/chrome/browser/extensions/extension_web_ui.cc b/chrome/browser/extensions/extension_web_ui.cc
|
| index d37df1d6e84b99cd241f9fc0c7c41070161baac1..e02a8e484d48f4dfd17370873cee190320d0283e 100644
|
| --- a/chrome/browser/extensions/extension_web_ui.cc
|
| +++ b/chrome/browser/extensions/extension_web_ui.cc
|
| @@ -131,9 +131,9 @@ ExtensionWebUI::ExtensionWebUI(content::WebUI* web_ui, const GURL& url)
|
| const Extension* extension =
|
| service->extensions()->GetExtensionOrAppByURL(ExtensionURLInfo(url));
|
| DCHECK(extension);
|
| - // Only hide the url for internal pages (e.g. chrome-extension or packaged
|
| - // component apps like bookmark manager.
|
| - bool should_hide_url = !extension->is_hosted_app();
|
| + // Only grey out the entire url for internal pages (e.g. chrome-extension or
|
| + // packaged component apps like bookmark manager).
|
| + bool should_grey_out_url = !extension->is_hosted_app();
|
|
|
| // The base class defaults to enabling WebUI bindings, but we don't need
|
| // those (this is also reflected in ChromeWebUIControllerFactory::
|
| @@ -150,13 +150,14 @@ ExtensionWebUI::ExtensionWebUI(content::WebUI* web_ui, const GURL& url)
|
| if (effective_url.host() == chrome::kChromeUINewTabHost) {
|
| web_ui->FocusLocationBarByDefault();
|
| } else {
|
| - // Current behavior of other chrome:// pages is to display the URL.
|
| - should_hide_url = false;
|
| + // Current behavior of other chrome:// pages is to emphasize the host,
|
| + // which means we shouldn't grey out the entire url.
|
| + should_grey_out_url = false;
|
| }
|
| }
|
|
|
| - if (should_hide_url)
|
| - web_ui->HideURL();
|
| + if (should_grey_out_url)
|
| + web_ui->GreyOutURL();
|
|
|
| web_ui->SetBindings(bindings);
|
|
|
|
|