WindowManagerClient::AddAccelerator() should take an array

services/ui/public/cpp/window_manager_delegate.h

Index: services/ui/public/cpp/window_manager_delegate.h
diff --git a/services/ui/public/cpp/window_manager_delegate.h b/services/ui/public/cpp/window_manager_delegate.h
index 2f724fc877fe9646be6ea52adc7f75ce30ea7b8d..b36cd8470c5ac3f527cbc35cf1bb53556ea690f7 100644
--- a/services/ui/public/cpp/window_manager_delegate.h
+++ b/services/ui/public/cpp/window_manager_delegate.h
@@ -15,6 +15,7 @@
 #include "base/callback_forward.h"
 #include "services/ui/public/interfaces/cursor.mojom.h"
 #include "services/ui/public/interfaces/event_matcher.mojom.h"
+#include "services/ui/public/interfaces/window_manager.mojom.h"
 #include "services/ui/public/interfaces/window_manager_constants.mojom.h"
 #include "services/ui/public/interfaces/window_tree_constants.mojom.h"
 #include "ui/events/mojo/event.mojom.h"
@@ -46,9 +47,9 @@ class WindowManagerClient {
   virtual void SetNonClientCursor(Window* window,
                                   mojom::Cursor non_client_cursor) = 0;
 
-  virtual void AddAccelerator(uint32_t id,
-                              mojom::EventMatcherPtr event_matcher,
-                              const base::Callback<void(bool)>& callback) = 0;
+  virtual void AddAccelerator(
+      mojo::Array<mojom::AcceleratorEventPtr> accelerator_event,

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

nit: Use plural form to make it a bit more clear what the var stands for:
AddAccelerators, accelerator_events, etc.

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

Done. I forgot to rename  this to multi_accelerators.

+      const base::Callback<void(bool)>& callback) = 0;
   virtual void RemoveAccelerator(uint32_t id) = 0;
   virtual void AddActivationParent(Window* window) = 0;
   virtual void RemoveActivationParent(Window* window) = 0;