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

Unified Diff: components/autofill/core/browser/autofill_metrics_unittest.cc

Issue 824513003: Standardize usage of virtual/override/final specifiers in components/. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Simplify Created 6 years 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: components/autofill/core/browser/autofill_metrics_unittest.cc
diff --git a/components/autofill/core/browser/autofill_metrics_unittest.cc b/components/autofill/core/browser/autofill_metrics_unittest.cc
index 20d99d894e355c0edef19acac9543024e142bcc4..554d7f3185dd958f40e74e8746385d1710d67304 100644
--- a/components/autofill/core/browser/autofill_metrics_unittest.cc
+++ b/components/autofill/core/browser/autofill_metrics_unittest.cc
@@ -45,7 +45,7 @@ class TestPersonalDataManager : public PersonalDataManager {
// Overridden to avoid a trip to the database. This should be a no-op except
// for the side-effect of logging the profile count.
- virtual void LoadProfiles() override {
+ void LoadProfiles() override {
std::vector<AutofillProfile*> profiles;
web_profiles_.release(&profiles);
WDResult<std::vector<AutofillProfile*> > result(AUTOFILL_PROFILES_RESULT,
@@ -55,15 +55,13 @@ class TestPersonalDataManager : public PersonalDataManager {
}
// Overridden to avoid a trip to the database.
- virtual void LoadCreditCards() override {}
+ void LoadCreditCards() override {}
void set_autofill_enabled(bool autofill_enabled) {
autofill_enabled_ = autofill_enabled;
}
- virtual bool IsAutofillEnabled() const override {
- return autofill_enabled_;
- }
+ bool IsAutofillEnabled() const override { return autofill_enabled_; }
private:
void CreateTestAutofillProfiles(ScopedVector<AutofillProfile>* profiles) {

Powered by Google App Engine
This is Rietveld 408576698