Topsites become keyedService based.

chrome/browser/history/chrome_history_client_factory.cc

 // Copyright 2014 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/history/chrome_history_client_factory.h"
 
 #include "base/memory/singleton.h"
 #include "chrome/browser/bookmarks/bookmark_model_factory.h"
 #include "chrome/browser/history/chrome_history_client.h"
+#include "chrome/browser/history/top_sites_service_factory.h"
 #include "chrome/browser/profiles/incognito_helpers.h"
 #include "chrome/browser/profiles/profile.h"
 #include "components/keyed_service/content/browser_context_dependency_manager.h"
 
 // static
 ChromeHistoryClient* ChromeHistoryClientFactory::GetForProfile(
     Profile* profile) {
   return static_cast<ChromeHistoryClient*>(
       GetInstance()->GetServiceForBrowserContext(profile, true));
 }
 
 // static
 ChromeHistoryClient* ChromeHistoryClientFactory::GetForProfileWithoutCreating(
     Profile* profile) {
   return static_cast<ChromeHistoryClient*>(
       GetInstance()->GetServiceForBrowserContext(profile, false));
 }
 
 // static
 ChromeHistoryClientFactory* ChromeHistoryClientFactory::GetInstance() {
   return Singleton<ChromeHistoryClientFactory>::get();
 }
 
 ChromeHistoryClientFactory::ChromeHistoryClientFactory()
     : BrowserContextKeyedServiceFactory(
           "ChromeHistoryClient",
           BrowserContextDependencyManager::GetInstance()) {
   DependsOn(BookmarkModelFactory::GetInstance());

[sdefresne, 2014/12/19 15:11:40]

You have to add an explicit dependency on TopSitesServiceFactory here so that
the KeyedService are build/destroyed in correct order. Please also add that
dependency to the factories of all other KeyedService that ends up using
TopSitesServiceFactory.

DependsOn(TopSitesServiceFactory::GetInstance());

[Jitu( very slow this week), 2014/12/26 14:04:27]

Done.

 }
 
 ChromeHistoryClientFactory::~ChromeHistoryClientFactory() {
 }
 
 KeyedService* ChromeHistoryClientFactory::BuildServiceInstanceFor(
     content::BrowserContext* context) const {
   Profile* profile = static_cast<Profile*>(context);
-  return new ChromeHistoryClient(BookmarkModelFactory::GetForProfile(profile),
-                                 profile,
-                                 profile->GetTopSites());
+  return new ChromeHistoryClient(
+      BookmarkModelFactory::GetForProfile(profile), profile,
+      TopSitesServiceFactory::GetForProfile(profile));
 }
 
 content::BrowserContext* ChromeHistoryClientFactory::GetBrowserContextToUse(
     content::BrowserContext* context) const {
   return chrome::GetBrowserContextRedirectedInIncognito(context);
 }
 
 bool ChromeHistoryClientFactory::ServiceIsNULLWhileTesting() const {
   return true;
 }