[Icons NTP] Refactor large_icon_source to extract the logic shared between desktop and Android to f…

chrome/browser/favicon/large_icon_service_factory.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.

[huangs, 2015/04/21 05:00:27]

NOTE: The left file was unfortunately matched against
  chrome_fallback_icon_client_factory.cc
but
  fallback_icon_service_factory.cc
would have been a better fit...

[beaudoin, 2015/04/21 15:11:11]

:( No idea what to do about this though...

 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "chrome/browser/favicon/chrome_fallback_icon_client_factory.h"
+#include "chrome/browser/favicon/large_icon_service_factory.h"
 
 #include "base/memory/singleton.h"
-#include "chrome/browser/favicon/chrome_fallback_icon_client.h"
-#include "chrome/browser/profiles/incognito_helpers.h"
+#include "chrome/browser/favicon/favicon_service_factory.h"
+#include "components/favicon/core/favicon_service.h"
+#include "components/favicon/core/large_icon_service.h"
 #include "components/keyed_service/content/browser_context_dependency_manager.h"
 #include "content/public/browser/browser_context.h"
 
-ChromeFallbackIconClientFactory::ChromeFallbackIconClientFactory()
-    : BrowserContextKeyedServiceFactory(
-        "ChromeFallbackIconClient",
-        BrowserContextDependencyManager::GetInstance()) {
-}
-
-ChromeFallbackIconClientFactory::~ChromeFallbackIconClientFactory() {
-}
-
 // static
-favicon::FallbackIconClient*
-ChromeFallbackIconClientFactory::GetForBrowserContext(
+favicon::LargeIconService* LargeIconServiceFactory::GetForBrowserContext(
     content::BrowserContext* context) {
-  return static_cast<favicon::FallbackIconClient*>(
+  return static_cast<favicon::LargeIconService*>(
       GetInstance()->GetServiceForBrowserContext(context, true));
 }
 
 // static
-ChromeFallbackIconClientFactory*
-ChromeFallbackIconClientFactory::GetInstance() {
-  return Singleton<ChromeFallbackIconClientFactory>::get();
+LargeIconServiceFactory* LargeIconServiceFactory::GetInstance() {
+  return Singleton<LargeIconServiceFactory>::get();
 }
 
-KeyedService* ChromeFallbackIconClientFactory::BuildServiceInstanceFor(
+LargeIconServiceFactory::LargeIconServiceFactory()
+    : BrowserContextKeyedServiceFactory(
+        "LargeIconService",
+        BrowserContextDependencyManager::GetInstance()) {
+  DependsOn(FaviconServiceFactory::GetInstance());
+}
+
+LargeIconServiceFactory::~LargeIconServiceFactory() {}
+
+KeyedService* LargeIconServiceFactory::BuildServiceInstanceFor(
     content::BrowserContext* context) const {
-  return new ChromeFallbackIconClient();
+  favicon::FaviconService* favicon_service =
+      FaviconServiceFactory::GetForBrowserContext(context);
+  return new favicon::LargeIconService(favicon_service);
 }
+
+bool LargeIconServiceFactory::ServiceIsNULLWhileTesting() const {
+  return true;
+}