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

Unified Diff: chrome/browser/ui/toolbar/app_menu_model.cc

Issue 2768633003: Dynamic updating recent menu for tabs from other devices. (Closed)
Patch Set: Created 3 years, 9 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
Index: chrome/browser/ui/toolbar/app_menu_model.cc
diff --git a/chrome/browser/ui/toolbar/app_menu_model.cc b/chrome/browser/ui/toolbar/app_menu_model.cc
index 7c6bd9250cb1d280b38cff0b57b0eccffad3279a..7dd6c98e868e2ba38a191bce55ab7e626ac6a5aa 100644
--- a/chrome/browser/ui/toolbar/app_menu_model.cc
+++ b/chrome/browser/ui/toolbar/app_menu_model.cc
@@ -717,9 +717,8 @@ void AppMenuModel::Build() {
AddSeparator(ui::NORMAL_SEPARATOR);
if (!browser_->profile()->IsOffTheRecord()) {
- recent_tabs_sub_menu_model_.reset(new RecentTabsSubMenuModel(provider_,
- browser_,
- NULL));
+ recent_tabs_sub_menu_model_.reset(
+ new RecentTabsSubMenuModel(provider_, browser_));
Peter Kasting 2017/03/23 04:48:34 Nit: While here, prefer MakeUnique to bare new:
AddSubMenuWithStringId(IDC_RECENT_TABS_MENU, IDS_HISTORY_MENU,
recent_tabs_sub_menu_model_.get());
}

Powered by Google App Engine
This is Rietveld 408576698