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

Unified Diff: chrome/browser/ui/views/bookmarks/bookmark_context_menu_controller_win.cc

Issue 242693003: Introduce BookmarkClient interface to abstract embedder (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 8 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/views/bookmarks/bookmark_context_menu_controller_win.cc
diff --git a/chrome/browser/ui/views/bookmarks/bookmark_context_menu_controller_win.cc b/chrome/browser/ui/views/bookmarks/bookmark_context_menu_controller_win.cc
index 5c1f8f99ee5707c4178437988a7dd862cf2ddde2..d55936f15f404f2e49de47b77efadb3abb3379a1 100644
--- a/chrome/browser/ui/views/bookmarks/bookmark_context_menu_controller_win.cc
+++ b/chrome/browser/ui/views/bookmarks/bookmark_context_menu_controller_win.cc
@@ -5,6 +5,8 @@
#include "chrome/browser/ui/bookmarks/bookmark_context_menu_controller.h"
#include "chrome/app/chrome_command_ids.h"
+#include "chrome/browser/bookmarks/bookmark_model.h"
+#include "chrome/browser/bookmarks/bookmark_model_factory.h"
#include "chrome/browser/bookmarks/bookmark_stats.h"
#include "chrome/browser/profiles/profile.h"
#include "chrome/browser/ui/bookmarks/bookmark_utils.h"
@@ -122,11 +124,12 @@ bool BookmarkContextMenuController::ExecutePlatformCommand(int command_id,
navigator = browser->tab_strip_model()->GetActiveWebContents();
}
+ BookmarkModel* model = BookmarkModelFactory::GetForProfile(profile_);
chrome::OpenAll(parent_window_, navigator, selection_,
NEW_FOREGROUND_TAB, profile_to_use);
for (size_t i = 0; i < selection_.size(); ++i) {
- RecordBookmarkLaunch(selection_[i],
- BOOKMARK_LAUNCH_LOCATION_CONTEXT_MENU);
+ RecordBookmarkLaunch(
+ model, selection_[i], BOOKMARK_LAUNCH_LOCATION_CONTEXT_MENU);
}
return true;
}

Powered by Google App Engine
This is Rietveld 408576698