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

Unified Diff: components/autofill/content/browser/content_autofill_driver.h

Issue 666133002: Standardize usage of virtual/override/final in components/ (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
« no previous file with comments | « no previous file | components/autofill/content/browser/content_autofill_driver_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/autofill/content/browser/content_autofill_driver.h
diff --git a/components/autofill/content/browser/content_autofill_driver.h b/components/autofill/content/browser/content_autofill_driver.h
index 50815933fa830a5a6161765d0b8bea98070d1578..8247c1dcc8dc0ad151575d94bdd1236da5edcd5d 100644
--- a/components/autofill/content/browser/content_autofill_driver.h
+++ b/components/autofill/content/browser/content_autofill_driver.h
@@ -42,23 +42,22 @@ class ContentAutofillDriver : public AutofillDriver,
static ContentAutofillDriver* FromWebContents(content::WebContents* contents);
// AutofillDriver:
- virtual bool IsOffTheRecord() const override;
- virtual net::URLRequestContextGetter* GetURLRequestContext() override;
- virtual base::SequencedWorkerPool* GetBlockingPool() override;
- virtual bool RendererIsAvailable() override;
- virtual void SendFormDataToRenderer(int query_id,
- RendererFormDataAction action,
- const FormData& data) override;
- virtual void PingRenderer() override;
- virtual void SendAutofillTypePredictionsToRenderer(
+ bool IsOffTheRecord() const override;
+ net::URLRequestContextGetter* GetURLRequestContext() override;
+ base::SequencedWorkerPool* GetBlockingPool() override;
+ bool RendererIsAvailable() override;
+ void SendFormDataToRenderer(int query_id,
+ RendererFormDataAction action,
+ const FormData& data) override;
+ void PingRenderer() override;
+ void SendAutofillTypePredictionsToRenderer(
const std::vector<FormStructure*>& forms) override;
- virtual void RendererShouldAcceptDataListSuggestion(
+ void RendererShouldAcceptDataListSuggestion(
const base::string16& value) override;
- virtual void RendererShouldClearFilledForm() override;
- virtual void RendererShouldClearPreviewedForm() override;
- virtual void RendererShouldFillFieldWithValue(
- const base::string16& value) override;
- virtual void RendererShouldPreviewFieldWithValue(
+ void RendererShouldClearFilledForm() override;
+ void RendererShouldClearPreviewedForm() override;
+ void RendererShouldFillFieldWithValue(const base::string16& value) override;
+ void RendererShouldPreviewFieldWithValue(
const base::string16& value) override;
AutofillExternalDelegate* autofill_external_delegate() {
@@ -73,16 +72,16 @@ class ContentAutofillDriver : public AutofillDriver,
AutofillClient* client,
const std::string& app_locale,
AutofillManager::AutofillDownloadManagerState enable_download_manager);
- virtual ~ContentAutofillDriver();
+ ~ContentAutofillDriver() override;
// content::WebContentsObserver:
- virtual void DidNavigateMainFrame(
+ void DidNavigateMainFrame(
const content::LoadCommittedDetails& details,
const content::FrameNavigateParams& params) override;
- virtual void NavigationEntryCommitted(
+ void NavigationEntryCommitted(
const content::LoadCommittedDetails& load_details) override;
- virtual void WasHidden() override;
- virtual bool OnMessageReceived(const IPC::Message& message) override;
+ void WasHidden() override;
+ bool OnMessageReceived(const IPC::Message& message) override;
// Sets the manager to |manager| and sets |manager|'s external delegate
// to |autofill_external_delegate_|. Takes ownership of |manager|.
« no previous file with comments | « no previous file | components/autofill/content/browser/content_autofill_driver_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698