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 80569d45c547444cf2af4e10e5fd3ec53519b48e..c0b01a2b14a5e5b7daf310a4efec1d8f13d290ea 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 AcceleratorEvent{

[mfomitchev, 2016/11/22 22:44:16]

"AcceleratorEvent" implies to me "event that triggered accelerator". This looks
like just an Accelerator, so call it that. Or AcceleratorTransport if you want
to have a different name than Accelerator in ui::ws

[thanhph, 2016/11/24 16:10:13]

Done, I use AcceleratorTransport to avoid name conflict/confusion during mass
renaming.

+   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);
+  AddAccelerator(array<AcceleratorEvent> multi_accelerators) => (bool success);
   RemoveAccelerator(uint32 id);
 
   // The window manager has completed a request with the specific change id.