Topsites become keyedService based.

chrome/browser/history/top_sites_provider.cc

Index: chrome/browser/history/top_sites_provider.cc
diff --git a/chrome/browser/history/top_sites_provider.cc b/chrome/browser/history/top_sites_provider.cc
new file mode 100644
index 0000000000000000000000000000000000000000..344914070a7099e38d5f08dde027a4e68c0acf1d
--- /dev/null
+++ b/chrome/browser/history/top_sites_provider.cc
@@ -0,0 +1,63 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.

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

Do not change copyright notice when renaming files.

+// Use of this source code is governed by a BSD-style license that can be

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

Why is this file not marked as moved but as new?

+// found in the LICENSE file.
+
+#include "chrome/browser/history/top_sites_provider.h"
+
+#include "base/logging.h"
+#include "base/strings/string_util.h"
+#include "chrome/browser/history/top_sites_service.h"
+#include "chrome/grit/chromium_strings.h"
+#include "chrome/grit/generated_resources.h"
+#include "chrome/grit/locale_settings.h"
+#include "grit/theme_resources.h"
+
+namespace history {
+
+const TopSitesProvider::PrepopulatedPage kPrepopulatedPages[] = {
+#if !defined(OS_ANDROID)
+    {IDS_CHROME_WELCOME_URL,
+     IDS_NEW_TAB_CHROME_WELCOME_PAGE_TITLE,
+     IDR_PRODUCT_LOGO_16,
+     IDR_NEWTAB_CHROME_WELCOME_PAGE_THUMBNAIL,
+     SkColorSetRGB(0, 147, 60)},
+    {IDS_WEBSTORE_URL,
+     IDS_EXTENSION_WEB_STORE_TITLE,
+     IDR_WEBSTORE_ICON_16,
+     IDR_NEWTAB_WEBSTORE_THUMBNAIL,
+     SkColorSetRGB(63, 132, 197)}
+#endif
+};
+
+// This function needs to remove
+// static
+TopSitesProvider* TopSitesProvider::Create(Profile* profile,

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

Remove this method, it should instead be the responsibility of the
TopSitesServiceFactory.

+                                           const base::FilePath& db_name) {
+  TopSitesService* top_sites_service = new TopSitesService(profile);
+  top_sites_service->Init(db_name);
+  return top_sites_service;
+}
+
+TopSitesProvider::TopSitesProvider() {

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

You define the constructor in the header and the implementation files. This
won't work.

+}
+
+TopSitesProvider::~TopSitesProvider() {
+}
+
+void TopSitesProvider::AddObserver(TopSitesObserver* observer) {
+  observer_list_.AddObserver(observer);
+}
+
+void TopSitesProvider::RemoveObserver(TopSitesObserver* observer) {
+  observer_list_.RemoveObserver(observer);
+}
+
+void TopSitesProvider::NotifyTopSitesLoaded() {
+  FOR_EACH_OBSERVER(TopSitesObserver, observer_list_, TopSitesLoaded(this));
+}
+
+void TopSitesProvider::NotifyTopSitesChanged() {
+  FOR_EACH_OBSERVER(TopSitesObserver, observer_list_, TopSitesChanged(this));
+}
+
+}  // namespace history