Make sure keybindings removed don't come back during extension update.

chrome/browser/extensions/api/commands/command_service.h

 // Copyright (c) 2012 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 CHROME_BROWSER_EXTENSIONS_API_COMMANDS_COMMAND_SERVICE_H_
 #define CHROME_BROWSER_EXTENSIONS_API_COMMANDS_COMMAND_SERVICE_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 91 matching lines @@
   // |allow_overrides| is false, the keybinding must be free for the change to
   // be recorded (as determined by the master list in |user_prefs|). If
   // |allow_overwrites| is true, any previously recorded keybinding for this
   // |accelerator| will be overwritten. Returns true if the change was
   // successfully recorded.
   bool AddKeybindingPref(const ui::Accelerator& accelerator,
                          std::string extension_id,
                          std::string command_name,
                          bool allow_overrides);
 
+  // Removes all keybindings for a given extension by its |extension_id|.
+  // |command_name| is optional and if specified, causes only the command with
+  // the name |command_name| to be removed.
+  void RemoveKeybindingPrefs(const std::string& extension_id,
+                             const std::string& command_name);
+
   // Update the keybinding prefs (for a command with a matching |extension_id|
   // and |command_name|) to |keystroke|. If the command had another key assigned
   // that key assignment will be removed.
   void UpdateKeybindingPrefs(const std::string& extension_id,
                              const std::string& command_name,
                              const std::string& keystroke);
 
   // Finds the shortcut assigned to a command with the name |command_name|
   // within an extension with id |extension_id|. Returns an empty Accelerator
   // object (with keycode VKEY_UNKNOWN) if no shortcut is assigned or the
@@ skipping 22 matching lines @@
     SCRIPT_BADGE,
   };
 
   // Assigns initial keybinding for a given |extension|'s page action, browser
   // action and named commands. In each case, if the suggested keybinding is
   // free, it will be taken by this extension. If not, that keybinding request
   // is ignored. |user_pref| is the PrefService used to record the new
   // keybinding assignment.
   void AssignInitialKeybindings(const extensions::Extension* extension);
 
-  // Removes all keybindings for a given extension by its |extension_id|.
-  // |command_name| is optional and if specified, causes only the command with
-  // the name |command_name| to be removed.
-  void RemoveKeybindingPrefs(const std::string& extension_id,
-                             const std::string& command_name);
-
   bool GetExtensionActionCommand(const std::string& extension_id,
                                  QueryType query_type,
                                  extensions::Command* command,
                                  bool* active,
                                  ExtensionActionType action_type);
 
   // The content notification registrar for listening to extension events.
   content::NotificationRegistrar registrar_;
 
   // A weak pointer to the profile we are associated with. Not owned by us.
   Profile* profile_;
 
   DISALLOW_COPY_AND_ASSIGN(CommandService);
 };
 
 }  //  namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_COMMANDS_COMMAND_SERVICE_H_