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

Unified Diff: chrome/browser/ui/views/keyboard_access_browsertest.cc

Issue 857433003: Update {virtual,override,final} to follow C++11 style chrome/browser/ui. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix rebase Created 5 years, 11 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: chrome/browser/ui/views/keyboard_access_browsertest.cc
diff --git a/chrome/browser/ui/views/keyboard_access_browsertest.cc b/chrome/browser/ui/views/keyboard_access_browsertest.cc
index cdcff2ac40040f09ce60464d4d7005b5540d3c6c..76636d51e908937360bc8bee2ff29a12f3d4359b 100644
--- a/chrome/browser/ui/views/keyboard_access_browsertest.cc
+++ b/chrome/browser/ui/views/keyboard_access_browsertest.cc
@@ -51,7 +51,7 @@ class ViewFocusChangeWaiter : public views::FocusChangeListener {
OnWillChangeFocus(NULL, focus_manager_->GetFocusedView());
}
- virtual ~ViewFocusChangeWaiter() {
+ ~ViewFocusChangeWaiter() override {
focus_manager_->RemoveFocusChangeListener(this);
}
@@ -61,12 +61,11 @@ class ViewFocusChangeWaiter : public views::FocusChangeListener {
private:
// Inherited from FocusChangeListener
- virtual void OnWillChangeFocus(views::View* focused_before,
- views::View* focused_now) override {
- }
+ void OnWillChangeFocus(views::View* focused_before,
+ views::View* focused_now) override {}
- virtual void OnDidChangeFocus(views::View* focused_before,
- views::View* focused_now) override {
+ void OnDidChangeFocus(views::View* focused_before,
+ views::View* focused_now) override {
if (focused_now && focused_now->id() != previous_view_id_) {
base::MessageLoop::current()->PostTask(FROM_HERE,
base::MessageLoop::QuitClosure());
@@ -90,7 +89,7 @@ class SendKeysMenuListener : public views::MenuListener {
toolbar_view_->AddMenuListener(this);
}
- virtual ~SendKeysMenuListener() {
+ ~SendKeysMenuListener() override {
if (test_dismiss_menu_)
toolbar_view_->RemoveMenuListener(this);
}
@@ -101,7 +100,7 @@ class SendKeysMenuListener : public views::MenuListener {
private:
// Overridden from views::MenuListener:
- virtual void OnMenuOpened() override {
+ void OnMenuOpened() override {
menu_open_count_++;
if (!test_dismiss_menu_) {
toolbar_view_->RemoveMenuListener(this);

Powered by Google App Engine
This is Rietveld 408576698