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

Unified Diff: athena/input/accelerator_manager_impl.cc

Issue 641683003: C++11 override style change for athena (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Created 6 years, 2 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 side-by-side diff with in-line comments
Download patch
Index: athena/input/accelerator_manager_impl.cc
diff --git a/athena/input/accelerator_manager_impl.cc b/athena/input/accelerator_manager_impl.cc
index bd6fbe636adb16dcc3bf763bcc0da7b424708adc..ecbae30c43d80da750fce2f222ee79abf0dc96a4 100644
--- a/athena/input/accelerator_manager_impl.cc
+++ b/athena/input/accelerator_manager_impl.cc
@@ -49,11 +49,11 @@ class NestedAcceleratorDelegate : public wm::NestedAcceleratorDelegate {
explicit NestedAcceleratorDelegate(
AcceleratorManagerImpl* accelerator_manager)
: accelerator_manager_(accelerator_manager) {}
- virtual ~NestedAcceleratorDelegate() {}
+ ~NestedAcceleratorDelegate() override {}
private:
// wm::NestedAcceleratorDelegate:
- virtual Result ProcessAccelerator(
+ Result ProcessAccelerator(
const ui::Accelerator& accelerator) override {
return accelerator_manager_->Process(accelerator) ? RESULT_PROCESSED
: RESULT_NOT_PROCESSED;
@@ -68,11 +68,11 @@ class AcceleratorDelegate : public wm::AcceleratorDelegate {
public:
explicit AcceleratorDelegate(AcceleratorManagerImpl* accelerator_manager)
: accelerator_manager_(accelerator_manager) {}
- virtual ~AcceleratorDelegate() {}
+ ~AcceleratorDelegate() override {}
private:
// wm::AcceleratorDelegate:
- virtual bool ProcessAccelerator(const ui::KeyEvent& event,
+ bool ProcessAccelerator(const ui::KeyEvent& event,
const ui::Accelerator& accelerator,
KeyType key_type) override {
aura::Window* target = static_cast<aura::Window*>(event.target());
@@ -92,13 +92,13 @@ class FocusManagerDelegate : public views::FocusManagerDelegate {
public:
explicit FocusManagerDelegate(AcceleratorManagerImpl* accelerator_manager)
: accelerator_manager_(accelerator_manager) {}
- virtual ~FocusManagerDelegate() {}
+ ~FocusManagerDelegate() override {}
- virtual bool ProcessAccelerator(const ui::Accelerator& accelerator) override {
+ bool ProcessAccelerator(const ui::Accelerator& accelerator) override {
return accelerator_manager_->Process(accelerator);
}
- virtual ui::AcceleratorTarget* GetCurrentTargetForAccelerator(
+ ui::AcceleratorTarget* GetCurrentTargetForAccelerator(
const ui::Accelerator& accelerator) const override {
return accelerator_manager_->IsRegistered(accelerator, AF_NONE)
? accelerator_manager_
@@ -119,9 +119,9 @@ class FocusManagerFactory : public views::FocusManagerFactory {
public:
explicit FocusManagerFactory(AcceleratorManagerImpl* accelerator_manager)
: accelerator_manager_(accelerator_manager) {}
- virtual ~FocusManagerFactory() {}
+ ~FocusManagerFactory() override {}
- virtual views::FocusManager* CreateFocusManager(
+ views::FocusManager* CreateFocusManager(
views::Widget* widget,
bool desktop_widget) override {
return new views::FocusManager(
@@ -140,7 +140,7 @@ class UIAcceleratorManagerWrapper
public:
UIAcceleratorManagerWrapper()
: ui_accelerator_manager_(new ui::AcceleratorManager) {}
- virtual ~UIAcceleratorManagerWrapper() {}
+ ~UIAcceleratorManagerWrapper() override {}
virtual void Register(const ui::Accelerator& accelerator,
ui::AcceleratorTarget* target) override {
@@ -172,7 +172,7 @@ class FocusManagerWrapper : public AcceleratorManagerImpl::AcceleratorWrapper {
public:
explicit FocusManagerWrapper(views::FocusManager* focus_manager)
: focus_manager_(focus_manager) {}
- virtual ~FocusManagerWrapper() {}
+ ~FocusManagerWrapper() override {}
virtual void Register(const ui::Accelerator& accelerator,
ui::AcceleratorTarget* target) override {

Powered by Google App Engine
This is Rietveld 408576698