Make mash register initial batch of accelerators in single shot.

services/ui/ws/window_tree.cc

 // Copyright 2014 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.
 
 #include "services/ui/ws/window_tree.h"
 
 #include <stddef.h>
 
 #include <utility>
 
@@ skipping 1742 matching lines @@
 }
 
 void WindowTree::AddAccelerators(
     std::vector<mojom::AcceleratorPtr> accelerators,
     const AddAcceleratorsCallback& callback) {
   DCHECK(window_manager_state_);
 
   bool success = true;
   for (auto iter = accelerators.begin(); iter != accelerators.end(); ++iter) {
     if (!window_manager_state_->event_dispatcher()->AddAccelerator(
-            iter->get()->id, std::move(iter->get()->event_matcher)))
+            iter->get()->id, std::move(iter->get()->event_matcher))) {
+      DVLOG(1) << "Can't add accelerator " << iter->get()->id;

[mfomitchev, 2017/01/03 19:54:35]

This isn't detailed enough - compare this to the original error message in
accelerator_controller_registrar.cc. Th "id" doesn't really help you debug as
much as keybode, type, and modifiers.
Also, can we move this to EventDispatcher::AddAccelerator and also log why it
failed (duplicated id vs. duplicate matcher)?

[thanhph1, 2017/01/06 21:55:56]

Done. I reverted this code and put logs in EventDispatcher::AddAccelerator
instead. Thanks!

       success = false;
+    }
   }
   callback.Run(success);
 }
 
 void WindowTree::RemoveAccelerator(uint32_t id) {
   window_manager_state_->event_dispatcher()->RemoveAccelerator(id);
 }
 
 void WindowTree::AddActivationParent(Id transport_window_id) {
   AddActivationParent(ClientWindowId(transport_window_id));
@@ skipping 241 matching lines @@
   client()->OnCompleteDrop(client_window_id.id, event_flags, cursor_offset,
                            effect_bitmask, callback);
 }
 
 void WindowTree::PerformOnDragDropDone() {
   client()->OnDragDropDone();
 }
 
 }  // namespace ws
 }  // namespace ui