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

Side by Side Diff: chrome/browser/tab_contents/render_view_context_menu.cc

Issue 9562002: Use a check item instead of changing its text. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 8 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 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 <algorithm> 5 #include <algorithm>
6 #include <set> 6 #include <set>
7 #include <utility> 7 #include <utility>
8 8
9 #include "chrome/browser/tab_contents/render_view_context_menu.h" 9 #include "chrome/browser/tab_contents/render_view_context_menu.h"
10 10
(...skipping 661 matching lines...) Expand 10 before | Expand all | Expand 10 after
672 void RenderViewContextMenu::LookUpInDictionary() { 672 void RenderViewContextMenu::LookUpInDictionary() {
673 // Used only in the Mac port. 673 // Used only in the Mac port.
674 NOTREACHED(); 674 NOTREACHED();
675 } 675 }
676 676
677 void RenderViewContextMenu::AddMenuItem(int command_id, 677 void RenderViewContextMenu::AddMenuItem(int command_id,
678 const string16& title) { 678 const string16& title) {
679 menu_model_.AddItem(command_id, title); 679 menu_model_.AddItem(command_id, title);
680 } 680 }
681 681
682 void RenderViewContextMenu::AddCheckItem(int command_id,
683 const string16& title) {
684 menu_model_.AddCheckItem(command_id, title);
685 }
686
682 void RenderViewContextMenu::AddSeparator() { 687 void RenderViewContextMenu::AddSeparator() {
683 menu_model_.AddSeparator(); 688 menu_model_.AddSeparator();
684 } 689 }
685 690
686 void RenderViewContextMenu::AddSubMenu(int command_id, 691 void RenderViewContextMenu::AddSubMenu(int command_id,
687 const string16& label, 692 const string16& label,
688 ui::MenuModel* model) { 693 ui::MenuModel* model) {
689 menu_model_.AddSubMenu(command_id, label, model); 694 menu_model_.AddSubMenu(command_id, label, model);
690 } 695 }
691 696
(...skipping 1298 matching lines...) Expand 10 before | Expand all | Expand 10 after
1990 source_web_contents_->GetRenderViewHost()-> 1995 source_web_contents_->GetRenderViewHost()->
1991 ExecuteMediaPlayerActionAtLocation(location, action); 1996 ExecuteMediaPlayerActionAtLocation(location, action);
1992 } 1997 }
1993 1998
1994 void RenderViewContextMenu::PluginActionAt( 1999 void RenderViewContextMenu::PluginActionAt(
1995 const gfx::Point& location, 2000 const gfx::Point& location,
1996 const WebPluginAction& action) { 2001 const WebPluginAction& action) {
1997 source_web_contents_->GetRenderViewHost()-> 2002 source_web_contents_->GetRenderViewHost()->
1998 ExecutePluginActionAtLocation(location, action); 2003 ExecutePluginActionAtLocation(location, action);
1999 } 2004 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698