Eliminate sending NOTIFICATION_TOP_SITES_* from TopSites

chrome/browser/history/top_sites_observer.h

Index: chrome/browser/history/top_sites_observer.h
diff --git a/chrome/browser/history/top_sites_observer.h b/chrome/browser/history/top_sites_observer.h
new file mode 100644
index 0000000000000000000000000000000000000000..0c24fa096f233816dff074322228a8bfaebe71fa
--- /dev/null
+++ b/chrome/browser/history/top_sites_observer.h
@@ -0,0 +1,24 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.

[sdefresne, 2014/08/11 12:01:31]

s/2013/2014/ and remove the (c). In general, you should use tools/boilerplate.py
to create new files as the script take care of creating correct headers, which
are (till the end of the year).

// 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.

[nshaik, 2014/08/13 05:40:45]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_HISTORY_TOP_SITES_OBSERVER_H_
+#define CHROME_BROWSER_HISTORY_TOP_SITES_OBSERVER_H_
+
+namespace history {
+class TopSites;
+
+// Interface for observing notifications from TopSites.
+class TopSitesObserver {
+ public:
+  // Is called when TopSites finishes loading.
+  virtual void TopSitesLoaded(history::TopSites* top_sites) = 0;
+
+  // Is called when TopSites when the either one of the most visited urls

[sdefresne, 2014/08/11 09:43:23]

nit: Please remove "when TopSites" and the extra "the" :-) 

// Is called when either one of the most visited urls change, or one of the
images changes.

[nshaik, 2014/08/13 05:40:45]

Sorry :) Silly mistake

+  // changed, or one of the images changes.
+  virtual void TopSitesChanged(history::TopSites* top_sites) = 0;
+};
+
+}  // namespace history
+
+#endif  // CHROME_BROWSER_HISTORY_TOP_SITES_OBSERVER_H_