Chromium Code Reviews| 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..b0566a48b2e2a46c1a84f8458b3589ed8aeb7551 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 don't emphasize host for internal pages (e.g. chrome-extension or |
| + // packaged component apps like bookmark manager. |
| + bool should_not_emphasize_host = !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,13 @@ 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 URL host. |
|
Devlin
2013/03/27 15:59:52
Don't add an extra :
Patrick Riordan
2013/03/27 19:43:28
Done.
|
| + should_not_emphasize_host = false; |
| } |
| } |
| - if (should_hide_url) |
| - web_ui->HideURL(); |
| + if (should_not_emphasize_host) |
| + web_ui->DontEmphasizeHost(); |
| web_ui->SetBindings(bindings); |