BookmarkManager keyboard shortcuts were broken

chrome/browser/views/task_manager_view.cc

 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/task_manager.h"
 
 #include "app/l10n_util.h"
 #include "app/table_model_observer.h"
 #include "base/stats_table.h"
 #include "chrome/app/chrome_dll_resource.h"
@@ skipping 197 matching lines @@
   virtual std::wstring GetWindowTitle() const;
   virtual std::wstring GetWindowName() const;
   virtual int GetDialogButtons() const;
   virtual void WindowClosing();
   virtual void DeleteDelegate();
   virtual views::View* GetContentsView();
 
   // views::TableViewObserver implementation.
   virtual void OnSelectionChanged();
   virtual void OnDoubleClick();
-  virtual void OnKeyDown(unsigned short virtual_keycode);
+  virtual void OnKeyDown(base::KeyboardCode keycode);
 
   // views::LinkController implementation.
   virtual void LinkActivated(views::Link* source, int event_flags);
 
   // Called by the column picker to pick up any new stat counters that
   // may have appeared since last time.
   void UpdateStatsCounters();
 
   // Menu::Delegate
   virtual void ShowContextMenu(views::View* source,
@@ skipping 346 matching lines @@
     }
   }
   kill_button_->SetEnabled(!selection_contains_browser_process &&
                            tab_table_->SelectedRowCount() > 0);
 }
 
 void TaskManagerView::OnDoubleClick() {
   ActivateFocusedTab();
 }
 
-void TaskManagerView::OnKeyDown(unsigned short virtual_keycode) {
-  if (virtual_keycode == base::VKEY_RETURN)
+void TaskManagerView::OnKeyDown(base::KeyboardCode keycode) {
+  if (keycode == base::VKEY_RETURN)
     ActivateFocusedTab();
 }
 
 // views::LinkController implementation
 void TaskManagerView::LinkActivated(views::Link* source, int event_flags) {
   DCHECK(source == about_memory_link_);
   task_manager_->OpenAboutMemory();
 }
 
 void TaskManagerView::ShowContextMenu(views::View* source, int x, int y,
@@ skipping 78 matching lines @@
 
 namespace browser {
 
 // Declared in browser_dialogs.h so others don't need to depend on our header.
 void ShowTaskManager() {
   TaskManagerView::Show();
 }
 
 }  // namespace browser