WindowManagerClient::AddAccelerator() should take an array

services/ui/common/accelerator_util.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef SERVICES_UI_COMMON_EVENT_MATCHER_UTIL_H_
-#define SERVICES_UI_COMMON_EVENT_MATCHER_UTIL_H_
+#ifndef SERVICES_UI_COMMON_ACCELERATOR_UTIL_H_
+#define SERVICES_UI_COMMON_ACCELERATOR_UTIL_H_
 
 #include "services/ui/public/interfaces/event_matcher.mojom.h"
+#include "services/ui/public/interfaces/window_manager.mojom.h"
 #include "ui/events/mojo/event_constants.mojom.h"
 #include "ui/events/mojo/keyboard_codes.mojom.h"
 
+namespace ash {

[mfomitchev, 2016/11/29 17:37:26]

This file is in ui/common, so this shouldn't be in ash::mus namespace, just put
it into ui namespace like CreateKeyMatcher. They are very close semantically -
that's why we are putting them into the same file.

[thanhph, 2016/11/29 19:06:41]

Done, thanks!

+namespace mus {
+
+// Construct accelerator from the provided |id| and |event_matcher| and add it
+// to an empty |accelerators| vector.
+std::vector<ui::mojom::AcceleratorTransportPtr> AddAcceleratorHelper(
+    uint32_t id,
+    ui::mojom::EventMatcherPtr event_matcher);
+
+}  // namespace mus
+}  // namespace ash
+
 namespace ui {
 
 // |flags| is a bitfield of kEventFlag* and kMouseEventFlag* values in
 // input_event_constants.mojom.
 mojom::EventMatcherPtr CreateKeyMatcher(ui::mojom::KeyboardCode code,
                                         int flags);
 
 }  // namespace ui
 
-#endif  // SERVICES_UI_COMMON_EVENT_MATCHER_UTIL_H_
+#endif  // SERVICES_UI_COMMON_ACCELERATOR_TRANSPORT_UTIL_H_

[mfomitchev, 2016/11/29 18:23:15]

update comment