Componentize TopSites, TopSitesBackend, TopSitesDatabase

chrome/browser/ui/webui/ntp/favicon_webui_handler.cc

Index: chrome/browser/ui/webui/ntp/favicon_webui_handler.cc
diff --git a/chrome/browser/ui/webui/ntp/favicon_webui_handler.cc b/chrome/browser/ui/webui/ntp/favicon_webui_handler.cc
index 20cfe1aadcfb132c1145a288e3068e70f53b2171..b1afc3ec5e7f50f61c7eb2b79308121682504b78 100644
--- a/chrome/browser/ui/webui/ntp/favicon_webui_handler.cc
+++ b/chrome/browser/ui/webui/ntp/favicon_webui_handler.cc
@@ -13,9 +13,10 @@
 #include "chrome/browser/extensions/extension_icon_manager.h"
 #include "chrome/browser/favicon/favicon_service.h"
 #include "chrome/browser/favicon/favicon_service_factory.h"
-#include "chrome/browser/history/top_sites.h"
+#include "chrome/browser/history/top_sites_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/url_constants.h"
+#include "components/history/core/browser/top_sites.h"
 #include "content/public/browser/web_ui.h"
 #include "extensions/browser/extension_registry.h"
 #include "third_party/skia/include/core/SkBitmap.h"
@@ -96,16 +97,19 @@ void FaviconWebUIHandler::HandleGetFaviconDominantColor(
     return;
 
   GURL url(path);
-  // Intercept requests for prepopulated pages.
-  for (size_t i = 0; i < history::kPrepopulatedPagesCount; i++) {
-    if (url.spec() ==
-        l10n_util::GetStringUTF8(history::kPrepopulatedPages[i].url_id)) {
-      base::StringValue dom_id_value(dom_id);
-      scoped_ptr<base::StringValue> color(
-          SkColorToCss(history::kPrepopulatedPages[i].color));
-      web_ui()->CallJavascriptFunction("ntp.setFaviconDominantColor",
-                                       dom_id_value, *color);
-      return;
+  // Intercept requests for prepopulated pages if TopSites exists.
+  scoped_refptr<history::TopSites> top_sites =
+      TopSitesFactory::GetForProfile(Profile::FromWebUI(web_ui()));
+  if (top_sites) {
+    for (const auto& prepopulated_page : top_sites->GetPrepopulatedPages()) {
+      if (url == prepopulated_page.url.url) {
+        base::StringValue dom_id_value(dom_id);
+        scoped_ptr<base::StringValue> color(
+            SkColorToCss(prepopulated_page.color));
+        web_ui()->CallJavascriptFunction("ntp.setFaviconDominantColor",
+                                         dom_id_value, *color);
+        return;
+      }
     }
   }