WindowManagerClient::AddAccelerator() should take an array

services/ui/public/interfaces/window_manager.mojom

Index: services/ui/public/interfaces/window_manager.mojom
diff --git a/services/ui/public/interfaces/window_manager.mojom b/services/ui/public/interfaces/window_manager.mojom
index c9d63f81da8d86a0243164be715647c4f4629350..9941992c6929a2fb032ab22e68783a1b466b004b 100644
--- a/services/ui/public/interfaces/window_manager.mojom
+++ b/services/ui/public/interfaces/window_manager.mojom
@@ -138,6 +138,11 @@ interface WindowManager {
   OnAccelerator(uint32 ack_id, uint32 accelerator_id, ui.mojom.Event event);
 };
 
+struct Accelerator {
+  uint32 id;
+  EventMatcher event_matcher;
+};
+
 // This interface is only used as an associated interface and is associated
 // with WindowTree.
 interface WindowManagerClient {
@@ -159,12 +164,13 @@ interface WindowManagerClient {
   // Add and remove accelerators. When accelerators are registered the
   // WindowManager receives the event via OnAccelerator() rather than the
   // target window. The id is defined by the client and can be used to more
-  // easily identify the accelerator's action. If an accelerator with the same
-  // id or the same matcher already exists, then the accelerator is not added.
+  // easily identify the accelerator's action.
   // Accelerator ids 1 << 31 and above are reserved for internal use.
-  //
+
   // See WindowTree for details on event dispatch.
-  AddAccelerator(uint32 id, EventMatcher matcher) => (bool success);
+  // If an accelerator with the same id or the same matcher already exists, skip

[sky, 2016/12/01 22:17:04]

How about: 'This ignores any accelerators already defined with the same id or
matcher.'

[thanhph, 2016/12/01 23:32:41]

Done, thanks!

+  // it. Returns true if all accelerators were added successfully.
+  AddAccelerators(array<Accelerator> accelerators) => (bool success);
   RemoveAccelerator(uint32 id);
 
   // The window manager has completed a request with the specific change id.