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

Unified Diff: chrome/browser/jumplist_win.h

Issue 845013002: Remove TopSites notification in favor of Observers (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/history/top_sites_impl_unittest.cc ('k') | chrome/browser/jumplist_win.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/jumplist_win.h
diff --git a/chrome/browser/jumplist_win.h b/chrome/browser/jumplist_win.h
index 0f9a7fb8c8545dd67d5bd71ec0d1ccc8431f45b4..84f8fadf52600f05df8c4453bca21099cd4bb4eb 100644
--- a/chrome/browser/jumplist_win.h
+++ b/chrome/browser/jumplist_win.h
@@ -22,6 +22,7 @@
#include "chrome/browser/sessions/tab_restore_service.h"
#include "chrome/browser/sessions/tab_restore_service_observer.h"
#include "components/history/core/browser/history_types.h"
+#include "components/history/core/browser/top_sites_observer.h"
#include "content/public/browser/browser_thread.h"
namespace chrome {
@@ -56,8 +57,10 @@ class Profile;
class JumpList : public TabRestoreServiceObserver,
public content::NotificationObserver,
public AvatarMenuObserver,
+ public history::TopSitesObserver,
public base::RefCountedThreadSafe<
- JumpList, content::BrowserThread::DeleteOnUIThread> {
+ JumpList,
+ content::BrowserThread::DeleteOnUIThread> {
public:
explicit JumpList(Profile* profile);
@@ -145,6 +148,10 @@ class JumpList : public TabRestoreServiceObserver,
// |profile_switcher_| ShellLinkItemList.
void UpdateProfileSwitcher();
+ // history::TopSitesObserver implementation.
+ void TopSitesLoaded(history::TopSites* top_sites) override;
+ void TopSitesChanged(history::TopSites* top_sites) override;
+
// Tracks FaviconService tasks.
base::CancelableTaskTracker cancelable_task_tracker_;
« no previous file with comments | « chrome/browser/history/top_sites_impl_unittest.cc ('k') | chrome/browser/jumplist_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698