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

Side by Side Diff: chrome/browser/bookmarks/bookmark_context_menu_controller.cc

Issue 10837317: Setting the touch wrench menu as default menu for ChromeOS (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: And another merge! Created 8 years, 4 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/bookmarks/bookmark_context_menu_controller.h" 5 #include "chrome/browser/bookmarks/bookmark_context_menu_controller.h"
6 6
7 #include "base/compiler_specific.h" 7 #include "base/compiler_specific.h"
8 #include "chrome/app/chrome_command_ids.h" 8 #include "chrome/app/chrome_command_ids.h"
9 #include "chrome/browser/bookmarks/bookmark_editor.h" 9 #include "chrome/browser/bookmarks/bookmark_editor.h"
10 #include "chrome/browser/bookmarks/bookmark_model.h" 10 #include "chrome/browser/bookmarks/bookmark_model.h"
(...skipping 81 matching lines...) Expand 10 before | Expand all | Expand 10 after
92 AddSeparator(); 92 AddSeparator();
93 AddItem(IDC_BOOKMARK_MANAGER, IDS_BOOKMARK_MANAGER); 93 AddItem(IDC_BOOKMARK_MANAGER, IDS_BOOKMARK_MANAGER);
94 AddCheckboxItem(IDC_BOOKMARK_BAR_ALWAYS_SHOW, IDS_SHOW_BOOKMARK_BAR); 94 AddCheckboxItem(IDC_BOOKMARK_BAR_ALWAYS_SHOW, IDS_SHOW_BOOKMARK_BAR);
95 } 95 }
96 96
97 void BookmarkContextMenuController::AddItem(int id, int localization_id) { 97 void BookmarkContextMenuController::AddItem(int id, int localization_id) {
98 menu_model_->AddItemWithStringId(id, localization_id); 98 menu_model_->AddItemWithStringId(id, localization_id);
99 } 99 }
100 100
101 void BookmarkContextMenuController::AddSeparator() { 101 void BookmarkContextMenuController::AddSeparator() {
102 menu_model_->AddSeparator(); 102 menu_model_->AddSeparator(ui::NORMAL_SEPARATOR);
103 } 103 }
104 104
105 void BookmarkContextMenuController::AddCheckboxItem(int id, 105 void BookmarkContextMenuController::AddCheckboxItem(int id,
106 int localization_id) { 106 int localization_id) {
107 menu_model_->AddCheckItemWithStringId(id, localization_id); 107 menu_model_->AddCheckItemWithStringId(id, localization_id);
108 } 108 }
109 109
110 void BookmarkContextMenuController::ExecuteCommand(int id) { 110 void BookmarkContextMenuController::ExecuteCommand(int id) {
111 if (delegate_) 111 if (delegate_)
112 delegate_->WillExecuteCommand(); 112 delegate_->WillExecuteCommand();
(...skipping 202 matching lines...) Expand 10 before | Expand all | Expand 10 after
315 delegate_->CloseMenu(); 315 delegate_->CloseMenu();
316 } 316 }
317 317
318 bool BookmarkContextMenuController::HasURLs() const { 318 bool BookmarkContextMenuController::HasURLs() const {
319 for (size_t i = 0; i < selection_.size(); ++i) { 319 for (size_t i = 0; i < selection_.size(); ++i) {
320 if (bookmark_utils::NodeHasURLs(selection_[i])) 320 if (bookmark_utils::NodeHasURLs(selection_[i]))
321 return true; 321 return true;
322 } 322 }
323 return false; 323 return false;
324 } 324 }
OLDNEW
« no previous file with comments | « chrome/browser/background/background_mode_manager.cc ('k') | chrome/browser/chromeos/status/network_menu.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698