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

Unified Diff: chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.h

Issue 671653002: Standardize usage of virtual/override/final in chrome/browser/ui/ (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/passwords/manage_passwords_ui_controller_mock.h
diff --git a/chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.h b/chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.h
index aad8d50f04dcf7b41ce51489099d93e412d00a29..cb001052e9ff980e8f063db2fbf47461ec8c57a0 100644
--- a/chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.h
+++ b/chrome/browser/ui/passwords/manage_passwords_ui_controller_mock.h
@@ -22,25 +22,25 @@ class ManagePasswordsUIControllerMock
public:
explicit ManagePasswordsUIControllerMock(
content::WebContents* contents);
- virtual ~ManagePasswordsUIControllerMock();
+ ~ManagePasswordsUIControllerMock() override;
// Navigation, surprisingly, is platform-specific; Android's settings page
// is native UI and therefore isn't available in a tab for unit tests.
//
// TODO(mkwst): Determine how to reasonably test this on that platform.
- virtual void NavigateToPasswordManagerSettingsPage() override;
+ void NavigateToPasswordManagerSettingsPage() override;
bool navigated_to_settings_page() const {
return navigated_to_settings_page_;
}
// We don't have a FormManager in tests, so stub these out.
- virtual void SavePasswordInternal() override;
+ void SavePasswordInternal() override;
bool saved_password() const { return saved_password_; }
- virtual void NeverSavePasswordInternal() override;
+ void NeverSavePasswordInternal() override;
bool never_saved_password() const { return never_saved_password_; }
- virtual const autofill::PasswordForm& PendingCredentials() const override;
+ const autofill::PasswordForm& PendingCredentials() const override;
void SetPendingCredentials(autofill::PasswordForm pending_credentials);
// Sneaky setters for testing.

Powered by Google App Engine
This is Rietveld 408576698