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

Unified Diff: chrome/browser/views/bookmark_context_menu_controller_views.cc

Issue 1795007: Get rid of BookmarkEditor::Show()'s BookmarkEditor::Handler argument. (Closed)
Patch Set: baz Created 10 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/views/bookmark_context_menu_controller_views.cc
diff --git a/chrome/browser/views/bookmark_context_menu_controller_views.cc b/chrome/browser/views/bookmark_context_menu_controller_views.cc
index 712df823a3340557637cf1397a34292eaa13e79c..8eda2139fdd73bc49b1908cc5670f00cb5b74822 100644
--- a/chrome/browser/views/bookmark_context_menu_controller_views.cc
+++ b/chrome/browser/views/bookmark_context_menu_controller_views.cc
@@ -157,7 +157,7 @@ void BookmarkContextMenuControllerViews::ExecuteCommand(int id) {
editor_config = BookmarkEditor::NO_TREE;
BookmarkEditor::Show(parent_window_, profile_, parent_,
BookmarkEditor::EditDetails(selection_[0]),
- editor_config, NULL);
+ editor_config);
} else {
BookmarkFolderEditorController::Show(profile_, parent_window_,
selection_[0], -1, BookmarkFolderEditorController::NONE);
@@ -189,7 +189,7 @@ void BookmarkContextMenuControllerViews::ExecuteCommand(int id) {
BookmarkEditor::Show(
parent_window_, profile_,
bookmark_utils::GetParentForNewNodes(parent_, selection_, NULL),
- BookmarkEditor::EditDetails(), editor_config, NULL);
+ BookmarkEditor::EditDetails(), editor_config);
break;
}

Powered by Google App Engine
This is Rietveld 408576698