Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(148)

Side by Side Diff: ui/base/accelerators/accelerator_manager_delegate.h

Issue 2586333003: Make mash register initial batch of accelerators in single shot. (Closed)
Patch Set: Add server log when adding accelerator vector. Fix/Refactor codes/comments/format. Created 3 years, 12 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef UI_BASE_ACCELERATORS_ACCELERATOR_MANAGER_DELEGATE_H_ 5 #ifndef UI_BASE_ACCELERATORS_ACCELERATOR_MANAGER_DELEGATE_H_
6 #define UI_BASE_ACCELERATORS_ACCELERATOR_MANAGER_DELEGATE_H_ 6 #define UI_BASE_ACCELERATORS_ACCELERATOR_MANAGER_DELEGATE_H_
7 7
8 #include "ui/base/ui_base_export.h" 8 #include "ui/base/ui_base_export.h"
9 9
10 namespace ui { 10 namespace ui {
11 11
12 class Accelerator; 12 class Accelerator;
13 13
14 class UI_BASE_EXPORT AcceleratorManagerDelegate { 14 class UI_BASE_EXPORT AcceleratorManagerDelegate {
15 public: 15 public:
16 // Called the first time a target is registered for |accelerator|. This is 16 // Called the first time a target is registered for |accelerator|. This is
17 // only called the first time a target is registered for a unique accelerator. 17 // only called the first time a target is registered for a unique accelerator.
18 // For example, if Register() is called twice with the same accelerator 18 // For example, if Register() is called twice with the same accelerator
19 // this is called only for the first call. 19 // this is called only for the first call.
20 virtual void OnAcceleratorRegistered(const Accelerator& accelerator) = 0; 20 virtual void OnAcceleratorRegistered(const Accelerator& accelerator) = 0;
21 21
22 // Called when there no more targets are registered for |accelerator|. 22 // Called when there no more targets are registered for |accelerator|.
23 virtual void OnAcceleratorUnregistered(const Accelerator& accelerator) = 0; 23 virtual void OnAcceleratorUnregistered(const Accelerator& accelerator) = 0;
24 24
25 // Called the first time a target is registered for each accelerator in
mfomitchev 2017/01/03 19:54:35 I think this would be a bit more clear: Called wh
thanhph1 2017/01/06 21:55:56 I modified it a bit. Let me know if it's clear. Th
26 // |accelerators|. This is only called the first time a target is registered
27 // for each unique accelerator. For example, if Register() is called twice
28 // with the same accelerator, this is called only for the first call.
29 virtual void OnAcceleratorsRegistered(
30 const std::vector<ui::Accelerator>& accelerators) = 0;
31
25 protected: 32 protected:
26 virtual ~AcceleratorManagerDelegate() {} 33 virtual ~AcceleratorManagerDelegate() {}
27 }; 34 };
28 35
29 } // namespace ui 36 } // namespace ui
30 37
31 #endif // UI_BASE_ACCELERATORS_ACCELERATOR_MANAGER_DELEGATE_H_ 38 #endif // UI_BASE_ACCELERATORS_ACCELERATOR_MANAGER_DELEGATE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698