Make mash register initial batch of accelerators in single shot.

services/ui/ws/window_tree.cc

Index: services/ui/ws/window_tree.cc
diff --git a/services/ui/ws/window_tree.cc b/services/ui/ws/window_tree.cc
index a9d098e8e875b8bb58e9885717c5aa61d75b0c71..5228a789543ab0058760baa0d024c077bc818e94 100644
--- a/services/ui/ws/window_tree.cc
+++ b/services/ui/ws/window_tree.cc
@@ -1760,8 +1760,10 @@ void WindowTree::AddAccelerators(
   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);
 }