From 71421c3fc725d9d55acba7a4233525f8b6e07350 Mon Sep 17 00:00:00 2001 From: "yutak@chromium.org" Date: Sat, 6 Jun 2009 00:41:44 +0000 Subject: Fix keyboard accelerator registration issue in views::View. View should not register the same accelerator target multiple times. BUG=13275 TEST=None Review URL: http://codereview.chromium.org/118242 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17810 0039d316-1c4b-4281-b951-d872f2087c98 --- views/focus/focus_manager.cc | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'views/focus/focus_manager.cc') diff --git a/views/focus/focus_manager.cc b/views/focus/focus_manager.cc index bab4922..04258bf 100644 --- a/views/focus/focus_manager.cc +++ b/views/focus/focus_manager.cc @@ -638,10 +638,8 @@ void FocusManager::RegisterAccelerator( const Accelerator& accelerator, AcceleratorTarget* target) { AcceleratorTargetList& targets = accelerators_[accelerator]; - // TODO(yutak): View::RegisterAccelerators() seems to register the same target - // multiple times. Should uncomment below after View is fixed. - // DCHECK(std::find(targets.begin(), targets.end(), target) == targets.end()) - // << "Registering the same target multiple times"; + DCHECK(std::find(targets.begin(), targets.end(), target) == targets.end()) + << "Registering the same target multiple times"; targets.push_front(target); } -- cgit v1.1