[Windows] Finish global and non-global media keys support on Windows.

chrome/browser/extensions/extension_keybinding_registry.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_EXTENSION_KEYBINDING_REGISTRY_H_
 #define CHROME_BROWSER_EXTENSIONS_EXTENSION_KEYBINDING_REGISTRY_H_
 
+#include <list>
 #include <map>
 #include <string>
 
 #include "base/compiler_specific.h"
 #include "content/public/browser/notification_details.h"
 #include "content/public/browser/notification_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "content/public/browser/notification_source.h"
 
 class Profile;
@@ skipping 59 matching lines @@
       const ui::Accelerator& accelerator,
       const std::string& command_name) = 0;
 
   // Make sure all extensions registered have keybindings added.
   void Init();
 
   // Whether to ignore this command. Only browserAction commands and pageAction
   // commands are currently ignored, since they are handled elsewhere.
   bool ShouldIgnoreCommand(const std::string& command) const;
 
+  // Fire event targets which the specified accelerator is binding with. Returns
+  // true if we can find the appropriate event targets.
+  bool NotifyEventTargetsByAccelerator(const ui::Accelerator& accelerator);

[Finnur, 2013/11/18 11:16:14]

You can probably skip the 'ByAccelerator' from the name.
Also, in the comment: s/accelerator/|accelerator|/ to indicate you are talking
about a param to the function.

[zhchbin, 2013/11/18 13:10:13]

Done.

+
   // Notifies appropriate parties that a command has been executed.
   void CommandExecuted(const std::string& extension_id,
                        const std::string& command);
 
-  // Maps an accelerator to a string pair (extension id, command name) for
-  // commands that have been registered. This keeps track of the targets for the
-  // keybinding event (which named command to call in which extension). On GTK,
-  // this map contains registration for pageAction and browserAction commands,
-  // whereas on other platforms it does not.
-  typedef std::map< ui::Accelerator,
-                    std::pair<std::string, std::string> > EventTargets;
+  // Maps an accelerator to a list of string pairs (extension id, command name)
+  // for commands that have been registered. This keeps track of the targets for
+  // the keybinding event (which named command to call in which extension). On
+  // GTK this map contains registration for pageAction and browserAction
+  // commands, whereas on other platforms it does not. Note that normal
+  // accelerator (which isn't media keys) has only one target, while the media
+  // keys can have more than one.
+  typedef std::list<std::pair<std::string, std::string> > TargetList;
+  typedef std::map<ui::Accelerator, TargetList> EventTargets;
   EventTargets event_targets_;
 
  private:
   // Returns true if the |extension| matches our extension filter.
   bool ExtensionMatchesFilter(const extensions::Extension* extension);
 
   // The content notification registrar for listening to extension events.
   content::NotificationRegistrar registrar_;
 
   // Weak pointer to our profile. Not owned by us.
   Profile* profile_;
 
   // What extensions to register keybindings for.
   ExtensionFilter extension_filter_;
 
   // Weak pointer to our delegate. Not owned by us. Must outlive this class.
   Delegate* delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(ExtensionKeybindingRegistry);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_EXTENSION_KEYBINDING_REGISTRY_H_