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

Unified Diff: chrome/browser/ui/views/task_manager_view.cc

Issue 9665008: views: Nuke is_mouse_gesture parameter from ContextMenuController::ShowContextMenuForView(). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: nuke is_mouse_gesture parameter 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/views/tabs/base_tab.cc ('k') | ui/views/context_menu_controller.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/task_manager_view.cc
diff --git a/chrome/browser/ui/views/task_manager_view.cc b/chrome/browser/ui/views/task_manager_view.cc
index 9d5fb9e3664e0d2f90762e3983b8c9cc85e7ddb7..8b51726fdc099bf7f2ca1238dee52f81b4b4db23 100644
--- a/chrome/browser/ui/views/task_manager_view.cc
+++ b/chrome/browser/ui/views/task_manager_view.cc
@@ -277,17 +277,17 @@ class TaskManagerView : public views::ButtonListener,
// resources in the resource display.
static void Show(bool highlight_background_resources);
- // views::View
+ // views::View:
virtual void Layout() OVERRIDE;
virtual gfx::Size GetPreferredSize() OVERRIDE;
virtual void ViewHierarchyChanged(bool is_add, views::View* parent,
views::View* child) OVERRIDE;
- // ButtonListener implementation.
+ // views::ButtonListener:
virtual void ButtonPressed(views::Button* sender,
const views::Event& event) OVERRIDE;
- // views::DialogDelegate
+ // views::DialogDelegateView:
virtual bool CanResize() const OVERRIDE;
virtual bool CanMaximize() const OVERRIDE;
virtual bool ExecuteWindowsCommand(int command_id) OVERRIDE;
@@ -297,24 +297,23 @@ class TaskManagerView : public views::ButtonListener,
virtual void WindowClosing() OVERRIDE;
virtual views::View* GetContentsView() OVERRIDE;
- // views::TableViewObserver implementation.
+ // views::TableViewObserver:
virtual void OnSelectionChanged() OVERRIDE;
virtual void OnDoubleClick() OVERRIDE;
virtual void OnKeyDown(ui::KeyboardCode keycode) OVERRIDE;
- // views::LinkListener implementation.
+ // views::LinkListener:
virtual void LinkClicked(views::Link* source, int event_flags) OVERRIDE;
// Called by the column picker to pick up any new stat counters that
// may have appeared since last time.
void UpdateStatsCounters();
- // views::ContextMenuController
+ // views::ContextMenuController:
virtual void ShowContextMenuForView(views::View* source,
- const gfx::Point& p,
- bool is_mouse_gesture) OVERRIDE;
+ const gfx::Point& point) OVERRIDE;
- // views::Menu::Delegate
+ // views::Menu::Delegate:
virtual bool IsItemChecked(int id) const OVERRIDE;
virtual void ExecuteCommand(int id) OVERRIDE;
@@ -708,8 +707,7 @@ void TaskManagerView::LinkClicked(views::Link* source, int event_flags) {
}
void TaskManagerView::ShowContextMenuForView(views::View* source,
- const gfx::Point& p,
- bool is_mouse_gesture) {
+ const gfx::Point& point) {
UpdateStatsCounters();
scoped_ptr<views::Menu> menu(views::Menu::Create(
this, views::Menu::TOPLEFT, source->GetWidget()->GetNativeView()));
@@ -718,7 +716,7 @@ void TaskManagerView::ShowContextMenuForView(views::View* source,
menu->AppendMenuItem(i->id, l10n_util::GetStringUTF16(i->id),
views::Menu::CHECKBOX);
}
- menu->RunMenuAt(p.x(), p.y());
+ menu->RunMenuAt(point.x(), point.y());
}
bool TaskManagerView::IsItemChecked(int id) const {
« no previous file with comments | « chrome/browser/ui/views/tabs/base_tab.cc ('k') | ui/views/context_menu_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698