Extension Commands changed by the user now take effect immediately.

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

 // 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.
 
 #include "chrome/browser/extensions/api/commands/command_service.h"
 
 #include "base/string_util.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_keybinding_registry.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/extensions/extension_system.h"
 #include "chrome/browser/prefs/scoped_user_pref_update.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_notification_types.h"
 #include "chrome/common/extensions/extension_manifest_constants.h"
 #include "chrome/common/pref_names.h"
+#include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_service.h"
 
 using extensions::Extension;
 
 namespace {
 
 const char kExtension[] = "extension";
 const char kCommandName[] = "command_name";
 
 std::string GetPlatformKeybindingKeyForAccelerator(
@@ skipping 86 matching lines @@
   std::string key = GetPlatformKeybindingKeyForAccelerator(accelerator);
 
   if (!allow_overrides && bindings->HasKey(key))
       return false;  // Already taken.
 
   DictionaryValue* keybinding = new DictionaryValue();
   keybinding->SetString(kExtension, extension_id);
   keybinding->SetString(kCommandName, command_name);
 
   bindings->Set(key, keybinding);
+
+  std::pair<const std::string, const std::string> details =
+      std::make_pair(extension_id, command_name);
+  content::NotificationService::current()->Notify(
+      chrome::NOTIFICATION_EXTENSION_COMMAND_ADDED,
+      content::Source<Profile>(profile_),
+      content::Details<
+      std::pair<const std::string, const std::string> >(&details));
+
   return true;
 }
 
 void CommandService::Observe(
     int type,
     const content::NotificationSource& source,
     const content::NotificationDetails& details) {
   switch (type) {
     case chrome::NOTIFICATION_EXTENSION_INSTALLED:
       AssignInitialKeybindings(
@@ skipping 103 matching lines @@
         if (command_name != command)
           continue;
       }
 
       keys_to_remove.push_back(key);
     }
   }
 
   for (KeysToRemove::const_iterator it = keys_to_remove.begin();
        it != keys_to_remove.end(); ++it) {
-    bindings->Remove(*it, NULL);
+    std::string key = *it;
+    bindings->Remove(key, NULL);
+
+    std::pair<const std::string, const std::string> details =
+        std::make_pair(extension_id, command_name);
+    content::NotificationService::current()->Notify(
+        chrome::NOTIFICATION_EXTENSION_COMMAND_REMOVED,
+        content::Source<Profile>(profile_),
+        content::Details<
+            std::pair<const std::string, const std::string> >(&details));
   }
 }
 
 bool CommandService::GetExtensionActionCommand(const std::string& extension_id,
                                                QueryType type,
                                                extensions::Command* command,
                                                bool* active,
                                                bool browser_action) {
   const ExtensionSet* extensions =
       ExtensionSystem::Get(profile_)->extension_service()->extensions();
@@ skipping 19 matching lines @@
     return false;
 
   *command = *requested_command;
   if (shortcut_assigned.key_code() != ui::VKEY_UNKNOWN)
     command->set_accelerator(shortcut_assigned);
 
   return true;
 }
 
 }  // namespace extensions