Clean up the WrenchMenuModel so that it uses SimpleMenu::Delegate.

app/menus/accelerator.h

 // Copyright (c) 2010 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.
 
 #ifndef APP_MENUS_ACCELERATOR_H_
 #define APP_MENUS_ACCELERATOR_H_
 #pragma once
 
 #include "base/keyboard_codes.h"
 
@@ skipping 50 matching lines @@
   }
 
  protected:
   // The keycode (VK_...).
   base::KeyboardCode key_code_;
 
   // The state of the Shift/Ctrl/Alt keys (platform-dependent).
   int modifiers_;
 };
 
+// Since acclerator code is one of the few things that can't be cross platform
+// in the chrome UI, separate out just the GetAcceleratorForCommandId() from
+// the menu delegates.
+class AcceleratorProvider {
+ public:
+  // Gets the accelerator for the specified command id. Returns true if the
+  // command id has a valid accelerator, false otherwise.
+  virtual bool GetAcceleratorForCommandId(
+      int command_id,
+      menus::Accelerator* accelerator) = 0;
+};

[sky, 2010/08/23 21:55:59]

virtual protected destructor?

+
 }
 
 #endif  // APP_MENUS_ACCELERATOR_H_