Make mash register initial batch of accelerators in single shot.

ui/base/accelerators/accelerator_manager_delegate.h

Index: ui/base/accelerators/accelerator_manager_delegate.h
diff --git a/ui/base/accelerators/accelerator_manager_delegate.h b/ui/base/accelerators/accelerator_manager_delegate.h
index e3c6d9ccf6e53ebee773c721a1374ae0b03e18c0..df55ecdacfa60135a60b092a443be91924603805 100644
--- a/ui/base/accelerators/accelerator_manager_delegate.h
+++ b/ui/base/accelerators/accelerator_manager_delegate.h
@@ -22,6 +22,12 @@ class UI_BASE_EXPORT AcceleratorManagerDelegate {
   // Called when there no more targets are registered for |accelerator|.
   virtual void OnAcceleratorUnregistered(const Accelerator& accelerator) = 0;
 
+  // Register all accelerators in |accelerators| vector with single IPC.

[mfomitchev, 2016/12/22 21:18:55]

This comment describes the behavior of implementation of
OnAcceleratorsRegistered in AcceleratorControllerRegistrar. Instead it should be
describing when OnAcceleratorsRegistered is called on the delegate
(AcceleratorManagerDelegate). Look at the comment for OnAcceleratorRegistered
and describe this one using a similar perspective.

[thanhph, 2016/12/24 18:52:16]

Done. I updated it to:

  // Called the first time a target is registered for each accelerator in
  // |accelerators|. This is only called the first time a target is registered
  // for each unique accelerator. For example, if Register() is called twice
  // with the same accelerator, this is called only for the first call.

+  virtual void OnAcceleratorsRegistered(
+      const std::vector<ui::Accelerator>& accelerators) = 0;
+  virtual void OnAcceleratorsUnregistered(
+      std::vector<ui::Accelerator>& accelerators) = 0;
+
  protected:
   virtual ~AcceleratorManagerDelegate() {}
 };