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..dd4fe7bbc26badfee4f462682e8369f076fada6e 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 AcceleratorTransport{
+   uint32 id;
+   EventMatcher event_matcher;
+};
+
 // This interface is only used as an associated interface and is associated
 // with WindowTree.
 interface WindowManagerClient {
@@ -164,7 +169,7 @@ interface WindowManagerClient {
   // 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);
+  AddAccelerators(array<AcceleratorTransport> accelerators) => (bool success);

[mfomitchev, 2016/11/29 01:55:46]

Document the semantics of the return value, since it's no longer trivial.

[thanhph, 2016/11/29 16:50:17]

Done.

   RemoveAccelerator(uint32 id);
 
   // The window manager has completed a request with the specific change id.