WindowManagerClient::AddAccelerator() should take an array

services/ui/ws/event_dispatcher.h

Index: services/ui/ws/event_dispatcher.h
diff --git a/services/ui/ws/event_dispatcher.h b/services/ui/ws/event_dispatcher.h
index 472a0c0e4038d6c0c1af45c7125ee3e1b876985e..968f51b4d10ca1caf029bd4579bbfdc4c5152533 100644
--- a/services/ui/ws/event_dispatcher.h
+++ b/services/ui/ws/event_dispatcher.h
@@ -14,7 +14,7 @@
 #include "base/macros.h"
 #include "services/ui/common/types.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/ws/drag_cursor_updater.h"
 #include "services/ui/ws/modal_window_controller.h"
 #include "services/ui/ws/server_window_observer.h"
@@ -126,6 +126,12 @@ class EventDispatcher : public ServerWindowObserver, public DragCursorUpdater {
   // already exists with the same id or the same matcher, then the accelerator
   // is not added. Returns whether adding the accelerator was successful or not.
   bool AddAccelerator(uint32_t id, mojom::EventMatcherPtr event_matcher);
+
+  // Adds the supplied accelerators and ignores any accelerators already
+  // defined with the same id or matcher. Returns true if all accelerators were
+  // added successfully.
+  bool AddAccelerators(std::vector<ui::mojom::AcceleratorPtr> accelerators);

[sky, 2016/12/02 00:11:16]

This should only be used from WindowTreeClient. Inline it there.

[thanhph, 2016/12/02 15:11:18]

Done. I added the comment right before this function.

+
   void RemoveAccelerator(uint32_t id);
 
   // Processes the supplied event, informing the delegate as approriate. This