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

Unified Diff: chrome/browser/ui/views/autofill/password_generation_popup_view_views.cc

Issue 686543002: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@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: chrome/browser/ui/views/autofill/password_generation_popup_view_views.cc
diff --git a/chrome/browser/ui/views/autofill/password_generation_popup_view_views.cc b/chrome/browser/ui/views/autofill/password_generation_popup_view_views.cc
index cf90a93b45632ed3e077bfe5a55f3dac2d803df0..964d3f65580811830b25854e2d2b33810471bc52 100644
--- a/chrome/browser/ui/views/autofill/password_generation_popup_view_views.cc
+++ b/chrome/browser/ui/views/autofill/password_generation_popup_view_views.cc
@@ -32,7 +32,7 @@ const int kHelpVerticalOffset = 5;
class PasswordTextBox : public views::View {
public:
PasswordTextBox() {}
- virtual ~PasswordTextBox() {}
+ ~PasswordTextBox() override {}
// |suggestion_text| prompts the user to select the password,
// |generated_password| is the generated password, and |font_list| is the font
@@ -62,7 +62,7 @@ class PasswordTextBox : public views::View {
}
// views::View:
- virtual bool CanProcessEventsWithinSubtree() const override {
+ bool CanProcessEventsWithinSubtree() const override {
// Send events to the parent view for handling.
return false;
}
@@ -78,7 +78,7 @@ class PasswordTextBox : public views::View {
class PasswordGenerationPopupViewViews::PasswordBox : public views::View {
public:
PasswordBox() {}
- virtual ~PasswordBox() {}
+ ~PasswordBox() override {}
// |password| is the generated password, |suggestion| is the text prompting
// the user to select the password, and |font_list| is the font used for all
@@ -112,12 +112,12 @@ class PasswordGenerationPopupViewViews::PasswordBox : public views::View {
}
// views::View:
- virtual bool CanProcessEventsWithinSubtree() const override {
+ bool CanProcessEventsWithinSubtree() const override {
// Send events to the parent view for handling.
return false;
}
- virtual void GetAccessibleState(ui::AXViewState* state) override {
+ void GetAccessibleState(ui::AXViewState* state) override {
state->role = ui::AX_ROLE_ALERT;
state->name = accessible_name_;
}

Powered by Google App Engine
This is Rietveld 408576698