Commit 3f914a68 authored by Jeevan Shikaram's avatar Jeevan Shikaram Committed by Commit Bot

[App Management] Set the favicon of App Management to be a gear.

The favicon for the App Management page was changed from the default favicon
to the favicon of the settings page.

Bug: 947895, 906508
Change-Id: Ibfec0f725024430ab883b52aa2231d810a326c2d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546741
Commit-Queue: Jeevan Shikaram <jshikaram@chromium.org>
Auto-Submit: Jeevan Shikaram <jshikaram@chromium.org>
Reviewed-by: default avatarDominick Ng <dominickn@chromium.org>
Reviewed-by: default avatarScott Chen <scottchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#646653}
parent d1a217bd
...@@ -897,6 +897,10 @@ base::RefCountedMemory* ChromeWebUIControllerFactory::GetFaviconResourceBytes( ...@@ -897,6 +897,10 @@ base::RefCountedMemory* ChromeWebUIControllerFactory::GetFaviconResourceBytes(
if (page_url.host_piece() == chrome::kChromeUIManagementHost) if (page_url.host_piece() == chrome::kChromeUIManagementHost)
return ManagementUI::GetFaviconResourceBytes(scale_factor); return ManagementUI::GetFaviconResourceBytes(scale_factor);
// Android doesn't use the App Management page.
if (page_url.host_piece() == chrome::kChromeUIAppManagementHost)
return settings_utils::GetFaviconResourceBytes(scale_factor);
#if BUILDFLAG(ENABLE_EXTENSIONS) #if BUILDFLAG(ENABLE_EXTENSIONS)
if (page_url.host_piece() == chrome::kChromeUIExtensionsHost) { if (page_url.host_piece() == chrome::kChromeUIExtensionsHost) {
return extensions::ExtensionsUI::GetFaviconResourceBytes(scale_factor); return extensions::ExtensionsUI::GetFaviconResourceBytes(scale_factor);
......
...@@ -22,6 +22,7 @@ const char kChromeUIAccessibilityHost[] = "accessibility"; ...@@ -22,6 +22,7 @@ const char kChromeUIAccessibilityHost[] = "accessibility";
const char kChromeUIAppIconHost[] = "app-icon"; const char kChromeUIAppIconHost[] = "app-icon";
const char kChromeUIAppIconURL[] = "chrome://app-icon/"; const char kChromeUIAppIconURL[] = "chrome://app-icon/";
const char kChromeUIAppLauncherPageHost[] = "apps"; const char kChromeUIAppLauncherPageHost[] = "apps";
const char kChromeUIAppManagementHost[] = "apps";
const char kChromeUIAppsURL[] = "chrome://apps/"; const char kChromeUIAppsURL[] = "chrome://apps/";
const char kChromeUIAutofillInternalsHost[] = "autofill-internals"; const char kChromeUIAutofillInternalsHost[] = "autofill-internals";
const char kChromeUIBluetoothInternalsHost[] = "bluetooth-internals"; const char kChromeUIBluetoothInternalsHost[] = "bluetooth-internals";
......
...@@ -29,6 +29,7 @@ extern const char kChromeUIAccessibilityHost[]; ...@@ -29,6 +29,7 @@ extern const char kChromeUIAccessibilityHost[];
extern const char kChromeUIAppIconHost[]; extern const char kChromeUIAppIconHost[];
extern const char kChromeUIAppIconURL[]; extern const char kChromeUIAppIconURL[];
extern const char kChromeUIAppLauncherPageHost[]; extern const char kChromeUIAppLauncherPageHost[];
extern const char kChromeUIAppManagementHost[];
extern const char kChromeUIAppsURL[]; extern const char kChromeUIAppsURL[];
extern const char kChromeUIAutofillInternalsHost[]; extern const char kChromeUIAutofillInternalsHost[];
extern const char kChromeUIBluetoothInternalsHost[]; extern const char kChromeUIBluetoothInternalsHost[];
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment