GTK: Query TopSites for the Most Visited pages and populate fill in the global History menu.

chrome/browser/ui/gtk/global_history_menu.cc

Index: chrome/browser/ui/gtk/global_history_menu.cc
diff --git a/chrome/browser/ui/gtk/global_history_menu.cc b/chrome/browser/ui/gtk/global_history_menu.cc
index 1fe561fa185a329336d5151f99b5115eb32cf580..3e4e516bfce19ce0344a06cb6dd1a63e274e9572 100644
--- a/chrome/browser/ui/gtk/global_history_menu.cc
+++ b/chrome/browser/ui/gtk/global_history_menu.cc
@@ -10,6 +10,7 @@
 #include "base/utf_string_conversions.h"
 #include "base/string_number_conversions.h"
 #include "chrome/browser/favicon_service.h"
+#include "chrome/browser/history/top_sites.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/sessions/tab_restore_service.h"
 #include "chrome/browser/sessions/tab_restore_service_factory.h"
@@ -95,6 +96,7 @@ class GlobalHistoryMenu::HistoryItem {
 GlobalHistoryMenu::GlobalHistoryMenu(Browser* browser)
     : browser_(browser),
       profile_(browser_->profile()),
+      top_sites_(NULL),
       default_favicon_(NULL),
       tab_restore_service_(NULL) {
 }
@@ -114,6 +116,16 @@ void GlobalHistoryMenu::Init(GtkWidget* history_menu) {
   default_favicon_ = GtkThemeService::GetDefaultFavicon(true);
 
   if (profile_) {
+    top_sites_ = profile_->GetTopSites();
+    if (top_sites_) {
+      GetTopSitesData();
+
+      // Register for notification when TopSites changes so that we can update
+      // ourself.
+      registrar_.Add(this, NotificationType::TOP_SITES_CHANGED,
+                     Source<history::TopSites>(top_sites_));
+    }
+
     tab_restore_service_ = TabRestoreServiceFactory::GetForProfile(profile_);
     if (tab_restore_service_) {
       tab_restore_service_->LoadTabsFromLastSession();
@@ -130,6 +142,42 @@ void GlobalHistoryMenu::Init(GtkWidget* history_menu) {
   }
 }
 
+void GlobalHistoryMenu::GetTopSitesData() {
+  if (top_sites_) {

[Evan Stade, 2011/04/28 22:38:47]

should this be a dcheck?

+    top_sites_->GetMostVisitedURLs(
+        &top_sites_consumer_,
+        NewCallback(this, &GlobalHistoryMenu::OnTopSitesReceived));
+  }
+}
+
+void GlobalHistoryMenu::OnTopSitesReceived(
+    const history::MostVisitedURLList& visited_list) {
+  ClearMenuSection(history_menu_, GlobalMenuBar::TAG_MOST_VISITED);
+
+  int index = GetIndexOfMenuItemWithTag(
+      history_menu_,
+      GlobalMenuBar::TAG_MOST_VISITED_HEADER) + 1;
+
+  for (size_t i = 0; i < visited_list.size(); ++i) {

[Evan Stade, 2011/04/28 22:38:47]

I think this list may be as long as 20; do we really want to show that many?

+    const history::MostVisitedURL& visited = visited_list[i];
+    if (visited.url.spec().empty())
+      break;  // This is the signal that there are no more real visited sites.
+
+    HistoryItem* item = new HistoryItem();
+    item->title = visited.title;
+    item->url = visited.url;
+
+    // The TopSites system doesn't give us icons; it gives us chrome:// urls to
+    // icons so fetch the icons normally.
+    GetFaviconForHistoryItem(item);
+
+    AddHistoryItemToMenu(item,
+                         history_menu_,
+                         GlobalMenuBar::TAG_MOST_VISITED,
+                         index++);
+  }
+}
+
 GlobalHistoryMenu::HistoryItem* GlobalHistoryMenu::HistoryItemForMenuItem(
     GtkWidget* menu_item) {
   MenuItemToHistoryMap::iterator it = menu_item_history_map_.find(menu_item);
@@ -318,12 +366,16 @@ void GlobalHistoryMenu::ClearMenuCallback(GtkWidget* menu_item,
 void GlobalHistoryMenu::Observe(NotificationType type,
                                 const NotificationSource& source,
                                 const NotificationDetails& details) {
-  DCHECK(type.value == NotificationType::BROWSER_THEME_CHANGED);
-
-  // Keeping track of which menu items have the default icon is going an
-  // error-prone pain, so instead just store the new default favicon and
-  // we'll update on the next menu change event.
-  default_favicon_ = GtkThemeService::GetDefaultFavicon(true);
+  if (type.value == NotificationType::BROWSER_THEME_CHANGED) {
+    // Keeping track of which menu items have the default icon is going an
+    // error-prone pain, so instead just store the new default favicon and
+    // we'll update on the next menu change event.
+    default_favicon_ = GtkThemeService::GetDefaultFavicon(true);
+  } else if (type.value == NotificationType::TOP_SITES_CHANGED) {
+    GetTopSitesData();
+  } else {
+    NOTREACHED();
+  }
 }
 
 void GlobalHistoryMenu::TabRestoreServiceChanged(TabRestoreService* service) {