summaryrefslogtreecommitdiffstats
path: root/chrome/browser/views/bookmark_context_menu.cc
diff options
context:
space:
mode:
authorsky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-01-28 19:36:46 +0000
committersky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-01-28 19:36:46 +0000
commitd1ba5b511c94fb02b79ceae67a172d6bdb716e98 (patch)
tree23288e2b27f8513e76e5f22750f3166c92edaae5 /chrome/browser/views/bookmark_context_menu.cc
parentd3038bf0175f75df75385a17298134a2ee39b8a1 (diff)
downloadchromium_src-d1ba5b511c94fb02b79ceae67a172d6bdb716e98.zip
chromium_src-d1ba5b511c94fb02b79ceae67a172d6bdb716e98.tar.gz
chromium_src-d1ba5b511c94fb02b79ceae67a172d6bdb716e98.tar.bz2
Makes it so deleting a bookmark from the context menu doesn't close
the bookmark menu. BUG=2469 TEST=click on a bookmark folder on the bookmark bar, right click on an item and chose delete. Make sure the folder stays up and still works correctly (and the item you deleted isn't there). Review URL: http://codereview.chromium.org/551178 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@37424 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/bookmark_context_menu.cc')
-rw-r--r--chrome/browser/views/bookmark_context_menu.cc18
1 files changed, 17 insertions, 1 deletions
diff --git a/chrome/browser/views/bookmark_context_menu.cc b/chrome/browser/views/bookmark_context_menu.cc
index a3f3a15..21eb200 100644
--- a/chrome/browser/views/bookmark_context_menu.cc
+++ b/chrome/browser/views/bookmark_context_menu.cc
@@ -25,7 +25,8 @@ BookmarkContextMenu::BookmarkContextMenu(
parent, selection,
configuration))),
parent_window_(parent_window),
- ALLOW_THIS_IN_INITIALIZER_LIST(menu_(new views::MenuItemView(this))) {
+ ALLOW_THIS_IN_INITIALIZER_LIST(menu_(new views::MenuItemView(this))),
+ observer_(NULL) {
controller_->BuildMenu();
}
@@ -56,6 +57,10 @@ bool BookmarkContextMenu::IsCommandEnabled(int command_id) const {
return controller_->IsCommandEnabled(command_id);
}
+bool BookmarkContextMenu::ShouldCloseAllMenusOnExecute(int id) {
+ return id != IDS_BOOKMARK_BAR_REMOVE;
+}
+
////////////////////////////////////////////////////////////////////////////////
// BookmarkContextMenu, BookmarkContextMenuControllerDelegate implementation:
@@ -79,3 +84,14 @@ void BookmarkContextMenu::AddCheckboxItem(int command_id) {
menu_->AppendMenuItem(command_id, l10n_util::GetString(command_id),
views::MenuItemView::CHECKBOX);
}
+
+void BookmarkContextMenu::WillRemoveBookmarks(
+ const std::vector<const BookmarkNode*>& bookmarks) {
+ if (observer_)
+ observer_->WillRemoveBookmarks(bookmarks);
+}
+
+void BookmarkContextMenu::DidRemoveBookmarks() {
+ if (observer_)
+ observer_->DidRemoveBookmarks();
+}