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..a8f0cb8f7c75e8806c5472745b36f2f65bca50a3 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 AddAccelerators(
+      std::vector<mojom::AcceleratorTransportPtr> multi_accelerators,

[mfomitchev, 2016/11/24 19:56:13]

I don't think "multi_" is providing value. The plural form already tells us that
there are many accelerators. Is there any reason we shouldn't rename this to
"accelerators"? Same goes to every other place "multi_" is used in this CL.

[thanhph, 2016/11/29 00:08:29]

Done.

+      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;