Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(68)

Side by Side Diff: chrome/browser/history/top_sites_observer.h

Issue 441623002: Eliminate sending NOTIFICATION_TOP_SITES_* from TopSites (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address review comments Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // 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 s
nshaik 2014/08/13 05:40:45 Done.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef CHROME_BROWSER_HISTORY_TOP_SITES_OBSERVER_H_
6 #define CHROME_BROWSER_HISTORY_TOP_SITES_OBSERVER_H_
7
8 namespace history {
9 class TopSites;
10
11 // Interface for observing notifications from TopSites.
12 class TopSitesObserver {
13 public:
14 // Is called when TopSites finishes loading.
15 virtual void TopSitesLoaded(history::TopSites* top_sites) = 0;
16
17 // 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 "
nshaik 2014/08/13 05:40:45 Sorry :) Silly mistake
18 // changed, or one of the images changes.
19 virtual void TopSitesChanged(history::TopSites* top_sites) = 0;
20 };
21
22 } // namespace history
23
24 #endif // CHROME_BROWSER_HISTORY_TOP_SITES_OBSERVER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698