Refactor extension app icon.

chrome/browser/extensions/extension_app_icon_service_factory.cc

Index: chrome/browser/extensions/extension_app_icon_service_factory.cc
diff --git a/chrome/browser/extensions/extension_app_icon_service_factory.cc b/chrome/browser/extensions/extension_app_icon_service_factory.cc
new file mode 100644
index 0000000000000000000000000000000000000000..b21c9eacee0afcdcfc58e7f010f865cf692a614e
--- /dev/null
+++ b/chrome/browser/extensions/extension_app_icon_service_factory.cc
@@ -0,0 +1,47 @@
+// Copyright 2017 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_app_icon_service_factory.h"
+
+#include "chrome/browser/extensions/extension_app_icon_service.h"
+#include "chrome/browser/profiles/incognito_helpers.h"
+#include "components/keyed_service/content/browser_context_dependency_manager.h"
+#include "extensions/browser/extension_registry_factory.h"
+
+namespace extensions {
+
+// static
+ExtensionAppIconService* ExtensionAppIconServiceFactory::GetForBrowserContext(
+    content::BrowserContext* context) {
+  return static_cast<ExtensionAppIconService*>(
+      GetInstance()->GetServiceForBrowserContext(context, true));
+}
+
+// static
+ExtensionAppIconServiceFactory* ExtensionAppIconServiceFactory::GetInstance() {
+  return base::Singleton<ExtensionAppIconServiceFactory>::get();
+}
+
+ExtensionAppIconServiceFactory::ExtensionAppIconServiceFactory()
+    : BrowserContextKeyedServiceFactory(
+          "ExtensionAppIconService",
+          BrowserContextDependencyManager::GetInstance()) {
+  DependsOn(ExtensionRegistryFactory::GetInstance());
+}
+
+ExtensionAppIconServiceFactory::~ExtensionAppIconServiceFactory() = default;
+
+KeyedService* ExtensionAppIconServiceFactory::BuildServiceInstanceFor(
+    content::BrowserContext* context) const {
+  return ExtensionAppIconService::Create(context);
+}
+
+content::BrowserContext* ExtensionAppIconServiceFactory::GetBrowserContextToUse(
+    content::BrowserContext* context) const {
+  // This matches the logic in ExtensionSyncServiceFactory, which uses the

[Devlin, 2017/04/26 22:41:57]

What's the pertinence of ExtensionSyncServiceFactory?

[khmel, 2017/04/27 00:18:12]

Removed. it seems it is redundant, came from copy/paste.

+  // original browser context.
+  return chrome::GetBrowserContextRedirectedInIncognito(context);
+}
+
+}  // namespace extensions