alternate ntp: add "Recent Tabs" submenu to wrench menu

chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc

+// Copyright 2012 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.
+
+#include "chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h"
+
+#include "base/bind.h"
+#include "base/string_number_conversions.h"
+#include "base/utf_string_conversions.h"
+#include "chrome/app/chrome_command_ids.h"
+#include "chrome/browser/favicon/favicon_service_factory.h"
+#include "chrome/browser/prefs/scoped_user_pref_update.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/browser/sessions/session_restore.h"
+#include "chrome/browser/sessions/tab_restore_service_delegate.h"
+#include "chrome/browser/sessions/tab_restore_service_factory.h"
+#include "chrome/browser/sync/glue/session_model_associator.h"
+#include "chrome/browser/sync/glue/synced_session.h"
+#include "chrome/browser/sync/profile_sync_service.h"
+#include "chrome/browser/sync/profile_sync_service_factory.h"
+#include "chrome/browser/ui/browser.h"
+#include "chrome/browser/ui/browser_commands.h"
+#include "chrome/browser/ui/browser_tabstrip.h"
+#include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "chrome/common/pref_names.h"
+#include "chrome/common/time_format.h"
+#include "chrome/common/url_constants.h"
+#include "grit/generated_resources.h"
+#include "grit/ui_resources.h"
+#include "ui/base/accelerators/accelerator.h"
+#include "ui/base/l10n/l10n_util.h"
+#include "ui/base/resource/resource_bundle.h"
+#include "ui/base/text/text_elider.h"
+#include "ui/gfx/favicon_size.h"
+
+#if defined(USE_ASH)
+#include "ash/accelerators/accelerator_table.h"
+#endif  // defined(USE_ASH)
+
+namespace {
+
+// First comamnd id for navigatable (and hence executable) tab menu item.
+// The model and menu are not 1-1:
+// - menu has "Reopen closed tab", "No tabs from other devices", device section
+//   headers and separtors
+// - model only has navigatabale (and hence executable) tab items.
+// Using an initial command id for tab items makes it easier and less
+// error-prone to manipulate the model and menu.
+// This value must be bigger than the maximum possible number of items in menu,
+// so that index of last menu item doesn't clash with this value when menu items
+// are retrieved via GetIndexOfCommandId.
+const int kFirstTabCommandId = 100;
+
+// Command Id for disabled menu items, e.g. device section header,
+// "No tabs from other devices", etc.
+const int kDisabledCommandId = 1000;
+
+// Comparator function for use with std::sort that will sort sessions by
+// descending modified_time (i.e., most recent first).
+bool SortSessionsByRecency(const browser_sync::SyncedSession* s1,
+                           const browser_sync::SyncedSession* s2) {
+  return s1->modified_time > s2->modified_time;
+}
+
+// Comparator function for use with std::sort that will sort tabs by
+// descending timestamp (i.e., most recent first).
+bool SortTabsByRecency(const SessionTab* t1, const SessionTab* t2) {
+  return t1->timestamp > t2->timestamp;
+}
+
+// Convert |model_index| to command id of menu item.
+int ModelIndexToCommandId(int model_index) {
+  int command_id = model_index + kFirstTabCommandId;
+  DCHECK(command_id != kDisabledCommandId);
+  return command_id;
+}
+
+// Convert |command_id| of menu item to index in model.
+int CommandIdToModelIndex(int command_id) {
+  DCHECK(command_id != kDisabledCommandId);
+  return command_id - kFirstTabCommandId;
+}
+
+}  // namepace
+
+// An element in |RecentTabsSubMenuModel::model_| that stores the navigation
+// information of a local or foreign tab required to restore the tab.
+struct RecentTabsSubMenuModel::NavigationItem {
+  NavigationItem() : tab_id(-1) {}
+
+  NavigationItem(const std::string& session_tag,
+                 const SessionID::id_type& tab_id,
+                 const GURL& url)
+      : session_tag(session_tag),
+        tab_id(tab_id),
+        url(url) {}
+
+  // For use by std::set for sorting.
+  bool operator<(const NavigationItem& other) const {
+    return url < other.url;
+  }
+
+  std::string session_tag;  // Empty for local tabs, non-empty for foreign tabs.
+  SessionID::id_type tab_id;  // -1 for invalid, >= 0 otherwise.
+  GURL url;
+};
+
+RecentTabsSubMenuModel::RecentTabsSubMenuModel(
+    ui::AcceleratorProvider* accelerator_provider,
+    Browser* browser)
+    : ALLOW_THIS_IN_INITIALIZER_LIST(ui::SimpleMenuModel(this)),
+      browser_(browser),
+      associator_(NULL),
+      ALLOW_THIS_IN_INITIALIZER_LIST(weak_ptr_factory_(this)) {
+  Build();
+
+  // Retrieve accelerator key for IDC_RESTORE_TAB now, because on ASH, it's not
+  // defined in |accelerator_provider|, but in shell, so simply retrieve it now
+  // for all ASH and non-ASH for use in |GetAcceleratorForCommandId|.
+#if defined(USE_ASH)
+  for (size_t i = 0; i < ash::kAcceleratorDataLength; ++i) {
+    const ash::AcceleratorData& accel_data = ash::kAcceleratorData[i];
+    if (accel_data.action == ash::RESTORE_TAB) {
+      reopen_closed_tab_accelerator_ = ui::Accelerator(accel_data.keycode,
+                                                       accel_data.modifiers);
+      break;
+    }
+  }
+#else
+  accelerator_provider->GetAcceleratorForCommandId(
+      IDC_RESTORE_TAB, &reopen_closed_tab_accelerator_);
+#endif  // defined(USE_ASH)
+}
+
+RecentTabsSubMenuModel::RecentTabsSubMenuModel(
+    ui::AcceleratorProvider* accelerator_provider,
+    Browser* browser,
+    browser_sync::SessionModelAssociator* associator,
+    bool setup_for_test)
+    : ALLOW_THIS_IN_INITIALIZER_LIST(ui::SimpleMenuModel(this)),
+      browser_(browser),
+      associator_(associator),
+      ALLOW_THIS_IN_INITIALIZER_LIST(weak_ptr_factory_(this)) {
+  DCHECK(associator_);
+  DCHECK(setup_for_test);

[sky, 2012/11/12 04:13:03]

Why the DCHECK for setup_for_test? If always true, why the boolean?

[kuan, 2012/11/12 13:24:26]

Done.  i was following what SessionModelAssociator did.

+  Build();
+}
+
+RecentTabsSubMenuModel::~RecentTabsSubMenuModel() {
+}
+
+bool RecentTabsSubMenuModel::IsCommandIdChecked(int command_id) const {
+  return false;
+}
+
+bool RecentTabsSubMenuModel::IsCommandIdEnabled(int command_id) const {
+  if (command_id == IDC_RESTORE_TAB)
+    return chrome::IsCommandEnabled(browser_, command_id);
+  if (command_id == kDisabledCommandId)
+    return false;
+  int model_index = CommandIdToModelIndex(command_id);
+  return model_index >= 0 && model_index < static_cast<int>(model_.size());
+}
+
+bool RecentTabsSubMenuModel::GetAcceleratorForCommandId(
+    int command_id, ui::Accelerator* accelerator) {
+  if (command_id == IDC_RESTORE_TAB &&
+      reopen_closed_tab_accelerator_.key_code() != ui::VKEY_UNKNOWN) {
+    *accelerator = reopen_closed_tab_accelerator_;
+    return true;
+  }
+  return false;
+}
+
+bool RecentTabsSubMenuModel::IsItemForCommandIdDynamic(int command_id) const {
+  return command_id == IDC_RESTORE_TAB;
+}
+
+string16 RecentTabsSubMenuModel::GetLabelForCommandId(int command_id) const {
+  DCHECK_EQ(command_id, IDC_RESTORE_TAB);
+
+  int string_id = IDS_RESTORE_TAB;
+  if (IsCommandIdEnabled(command_id)) {
+    TabRestoreService* service =
+        TabRestoreServiceFactory::GetForProfile(browser_->profile());
+    if (service &&

[sky, 2012/11/12 04:13:03]

!service->entries().empty()

[kuan, 2012/11/12 13:24:26]

Done.

+        service->entries().front()->type == TabRestoreService::WINDOW) {
+      string_id = IDS_RESTORE_WINDOW;
+    }
+  }
+  return l10n_util::GetStringUTF16(string_id);
+}
+
+void RecentTabsSubMenuModel::ExecuteCommand(int command_id) {
+  ExecuteCommand(command_id, 0);
+}
+
+void RecentTabsSubMenuModel::ExecuteCommand(int command_id, int event_flags) {
+  if (command_id == IDC_RESTORE_TAB) {
+    chrome::ExecuteCommandWithDisposition(browser_, command_id,
+        chrome::DispositionFromEventFlags(event_flags));
+    return;
+  }
+
+  DCHECK(command_id != kDisabledCommandId);
+  int model_idx = CommandIdToModelIndex(command_id);
+  DCHECK(model_idx >= 0 && model_idx < static_cast<int>(model_.size()));
+  const NavigationItem& item = model_[model_idx];
+  DCHECK(item.tab_id > -1 && item.url.is_valid());
+
+  WindowOpenDisposition disposition =
+      chrome::DispositionFromEventFlags(event_flags);
+  if (disposition == CURRENT_TAB)  // Force to open a new foreground tab.
+    disposition = NEW_FOREGROUND_TAB;
+
+  if (item.session_tag.empty()) {  // Restore tab of local session.
+    TabRestoreService* service =
+        TabRestoreServiceFactory::GetForProfile(browser_->profile());
+    if (!service)
+      return;
+    TabRestoreServiceDelegate* delegate =
+        TabRestoreServiceDelegate::FindDelegateForWebContents(
+            chrome::GetActiveWebContents(browser_));
+    if (!delegate)
+      return;
+    service->RestoreEntryById(delegate, item.tab_id, disposition);
+  } else {  // Restore tab of foreign session.
+    browser_sync::SessionModelAssociator* associator = GetModelAssociator();
+    if (!associator)
+      return;
+    const SessionTab* tab;
+    if (!associator->GetForeignTab(item.session_tag, item.tab_id, &tab))
+      return;
+    if (tab->navigations.empty())
+      return;
+    int prev_num_tabs = browser_->tab_strip_model()->count();
+    SessionRestore::RestoreForeignSessionTab(
+        chrome::GetActiveWebContents(browser_), *tab, disposition);
+    // If tab is successfully added (i.e. there's 1 more tab), select it.
+    if (browser_->tab_strip_model()->count() == prev_num_tabs + 1)
+      chrome::SelectNextTab(browser_);
+  }
+}
+
+void RecentTabsSubMenuModel::Build() {
+  // The menu contains:
+  // - Reopen closed tab, then separator
+  // - device 1 section header, then list of tabs from device, then separator
+  // - device 2 section header, then list of tabs from device, then separator
+  // ...
+  // |model_| only contains navigatable (and hence executable) tab items for
+  // other devices.
+  BuildLastClosed();
+  BuildDevices();
+  if (model_.empty())
+    AddItemWithStringId(kDisabledCommandId, IDS_RECENT_TABS_NO_DEVICE_TABS);
+}
+
+void RecentTabsSubMenuModel::BuildLastClosed() {
+  AddItem(IDC_RESTORE_TAB, GetLabelForCommandId(IDC_RESTORE_TAB));
+  AddSeparator(ui::NORMAL_SEPARATOR);
+}
+
+void RecentTabsSubMenuModel::BuildDevices() {
+  browser_sync::SessionModelAssociator* associator = GetModelAssociator();
+  if (!associator)
+    return;
+
+  std::vector<const browser_sync::SyncedSession*> sessions;
+  if (!associator->GetAllForeignSessions(&sessions))
+    return;
+
+  // Sort sessions from most recent to least recent.
+  std::sort(sessions.begin(), sessions.end(), SortSessionsByRecency);
+
+  const size_t kMaxSessionsToShow = 3;
+  size_t num_sessions_added = 0;
+  bool need_separator = false;
+  for (size_t i = 0;
+       i < sessions.size() && num_sessions_added < kMaxSessionsToShow;
+       ++i) {
+    const browser_sync::SyncedSession* session = sessions[i];
+    const std::string& session_tag = session->session_tag;
+
+    // Get windows of session.
+    std::vector<const SessionWindow*> windows;
+    if (!associator->GetForeignSession(session_tag, &windows) ||
+        windows.empty()) {
+      continue;
+    }
+
+    // Collect tabs from all windows of session, pruning those that are not
+    // syncable or are NewTabPage, then sort them from most recent to least
+    // recent, independent of which window the tabs were from.
+    std::vector<const SessionTab*> tabs_in_session;
+    for (size_t j = 0; j < windows.size(); ++j) {
+      const SessionWindow* window = windows[j];
+      for (size_t t = 0; t < window->tabs.size(); ++t) {
+        const SessionTab* tab = window->tabs[t];
+        if (tab->navigations.empty())
+          continue;
+        const TabNavigation& current_navigation =
+            tab->navigations.at(tab->normalized_navigation_index());
+        if (current_navigation.virtual_url() ==
+            GURL(chrome::kChromeUINewTabURL)) {
+          continue;
+        }
+        tabs_in_session.push_back(tab);
+      }
+    }
+    if (tabs_in_session.empty())
+      continue;
+    std::sort(tabs_in_session.begin(), tabs_in_session.end(),
+              SortTabsByRecency);
+
+    // Build tab menu items from sorted session tabs.
+    const size_t kMaxTabsPerSessionToShow = 4;
+    for (size_t k = 0;
+         k < std::min(tabs_in_session.size(), kMaxTabsPerSessionToShow);
+         ++k) {
+      BuildForeignTabItem(session_tag, *tabs_in_session[k],
+          // Only need |session_name| for the first tab of the session.
+          !k ? session->session_name : std::string(), need_separator);
+      need_separator = false;
+    }  // for all tabs in one session
+
+    ++num_sessions_added;
+    need_separator = true;
+  }  // for all sessions
+}
+
+void RecentTabsSubMenuModel::BuildForeignTabItem(
+    const std::string& session_tag,
+    const SessionTab& tab,
+    const std::string& session_name,
+    bool need_separator) {
+  if (need_separator)
+    AddSeparator(ui::NORMAL_SEPARATOR);
+
+  if (!session_name.empty())
+    AddItem(kDisabledCommandId, UTF8ToUTF16(session_name));
+
+  const TabNavigation& current_navigation =
+      tab.navigations.at(tab.normalized_navigation_index());
+  NavigationItem item(session_tag, tab.tab_id.id(),
+                      current_navigation.virtual_url());
+  const int kMaxTabTitleWidth = 320;

[sky, 2012/11/12 04:13:03]

How come you need this? Does the menu handle it for you?

[kuan, 2012/11/12 13:24:26]

i thought this would fix the text and menu item width.  apparently, it only
fixed the text length, but not the menu item width.
while awaiting review for this cl, i started to look into fixing the menu width
in the next cl.  i think it's via WrenchMenu::GetMaxWidthForMenu, but haven't 
confirmed it.  the change doesn't look trivial, especially for this big cl.  so
i've removed elide text for now and will handle it in the next cl.

+  string16 tab_title = ui::ElideText(current_navigation.title(), gfx::Font(),
+                                     kMaxTabTitleWidth, ui::ELIDE_AT_END);
+  int command_id = ModelIndexToCommandId(model_.size());
+  AddItem(command_id, tab_title);
+  AddFavicon(model_.size(), command_id, item.url);
+  model_.push_back(item);
+}
+
+void RecentTabsSubMenuModel::AddFavicon(int model_index, int command_id,
+                                        const GURL& url) {
+  if (default_favicon_.IsEmpty()) {

[sky, 2012/11/12 04:13:03]

nit: move to constructor so you don't need to check each time through.

[kuan, 2012/11/12 13:24:26]

Done.

+    ResourceBundle& rb = ResourceBundle::GetSharedInstance();
+    default_favicon_ = rb.GetNativeImageNamed(IDR_DEFAULT_FAVICON);
+  }
+  // Set default icon first.
+  SetIcon(GetIndexOfCommandId(command_id), default_favicon_);
+  // Start request to fetch actual icon if possible.
+  FaviconService* favicon_service = FaviconServiceFactory::GetForProfile(
+      browser_->profile(), Profile::EXPLICIT_ACCESS);
+  if (!favicon_service)
+    return;
+  FaviconService::Handle handle = favicon_service->GetFaviconImageForURL(
+      FaviconService::FaviconForURLParams(browser_->profile(), url,
+          history::FAVICON, gfx::kFaviconSize, &favicon_consumer_),
+      base::Bind(&RecentTabsSubMenuModel::OnFaviconDataAvailable,
+                 weak_ptr_factory_.GetWeakPtr()));
+  favicon_consumer_.SetClientData(favicon_service, handle, command_id);
+}
+
+void RecentTabsSubMenuModel::OnFaviconDataAvailable(
+    FaviconService::Handle handle,
+    const history::FaviconImageResult& image_result) {
+  if (image_result.image.IsEmpty())
+    return;
+  DCHECK(!model_.empty());
+  int command_id = favicon_consumer_.GetClientData(
+      FaviconServiceFactory::GetForProfile(browser_->profile(),
+                                           Profile::EXPLICIT_ACCESS),
+      handle);
+  int index_in_menu = GetIndexOfCommandId(command_id);
+  DCHECK(index_in_menu != -1);
+  SetIcon(index_in_menu, image_result.image);
+  if (menu_model_delegate())
+    menu_model_delegate()->OnIconChanged(index_in_menu);
+}
+
+browser_sync::SessionModelAssociator*
+    RecentTabsSubMenuModel::GetModelAssociator() {
+  if (!associator_) {
+    ProfileSyncService* service = ProfileSyncServiceFactory::GetInstance()->
+        GetForProfile(browser_->profile());
+    // Only return the associator if it exists and it is done syncing sessions.
+    if (service && service->ShouldPushChanges())
+      associator_ = service->GetSessionModelAssociator();
+  }
+  return associator_;
+}