Make mash register initial batch of accelerators in single shot.

ui/base/accelerators/accelerator_manager_delegate.h

 // Copyright 2016 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 UI_BASE_ACCELERATORS_ACCELERATOR_MANAGER_DELEGATE_H_
 #define UI_BASE_ACCELERATORS_ACCELERATOR_MANAGER_DELEGATE_H_
 
 #include "ui/base/ui_base_export.h"
 
 namespace ui {
 
 class Accelerator;
 
 class UI_BASE_EXPORT AcceleratorManagerDelegate {
  public:
   // Called the first time a target is registered for |accelerator|. This is
   // only called the first time a target is registered for a unique accelerator.
   // For example, if Register() is called twice with the same accelerator
   // this is called only for the first call.
   virtual void OnAcceleratorRegistered(const Accelerator& accelerator) = 0;

[mfomitchev, 2017/01/24 22:56:03]

This doesn't actually seem to be called anywhere anymore?

[thanhph1, 2017/01/25 20:12:37]

This is used in accelerator_controller_registrar.cc (line 119) and in
accelerator_manager_unittest.cc

[mfomitchev, 2017/01/26 16:03:36]

We don't call this method in accelerator_controller_registrar.cc - we override
it. Same with the unit test.
(Worth noting that even if the unit test did call the method - that wouldn't be
very relevant - usually it would be due to the fact that the unit test is
testing functionality we don't use. Either way, we shouldn't have test-only
method in a public interface used by production code like this one).

[thanhph1, 2017/01/26 19:33:38]

Cool, thanks! I removed this function.

 
   // Called when there no more targets are registered for |accelerator|.
   virtual void OnAcceleratorUnregistered(const Accelerator& accelerator) = 0;
 
+  // Called when a number of accelerators are registered at once. An accelerator
+  // is only included in the |accelerators| vector when the first time a target
+  // for the accelerator is registered. For example, if Register() is called
+  // twice with the same accelerator, this is called only for the first call.
+  virtual void OnAcceleratorVectorRegistered(
+      const std::vector<ui::Accelerator>& accelerators) = 0;
+
  protected:
   virtual ~AcceleratorManagerDelegate() {}
 };
 
 }  // namespace ui
 
 #endif  // UI_BASE_ACCELERATORS_ACCELERATOR_MANAGER_DELEGATE_H_