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

Unified Diff: components/dom_distiller/content/distiller_page_web_contents.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
Index: components/dom_distiller/content/distiller_page_web_contents.h
diff --git a/components/dom_distiller/content/distiller_page_web_contents.h b/components/dom_distiller/content/distiller_page_web_contents.h
index a5a46c318a10b44a531c494875c0ce20600b655d..e73b0ccc9b04e5780664768f3189e76507e171b6 100644
--- a/components/dom_distiller/content/distiller_page_web_contents.h
+++ b/components/dom_distiller/content/distiller_page_web_contents.h
@@ -20,7 +20,7 @@ class SourcePageHandleWebContents : public SourcePageHandle {
public:
explicit SourcePageHandleWebContents(
scoped_ptr<content::WebContents> web_contents);
- virtual ~SourcePageHandleWebContents();
+ ~SourcePageHandleWebContents() override;
scoped_ptr<content::WebContents> GetWebContents();
@@ -34,11 +34,11 @@ class DistillerPageWebContentsFactory : public DistillerPageFactory {
explicit DistillerPageWebContentsFactory(
content::BrowserContext* browser_context)
: DistillerPageFactory(), browser_context_(browser_context) {}
- virtual ~DistillerPageWebContentsFactory() {}
+ ~DistillerPageWebContentsFactory() override {}
- virtual scoped_ptr<DistillerPage> CreateDistillerPage(
+ scoped_ptr<DistillerPage> CreateDistillerPage(
const gfx::Size& render_view_size) const override;
- virtual scoped_ptr<DistillerPage> CreateDistillerPageWithHandle(
+ scoped_ptr<DistillerPage> CreateDistillerPageWithHandle(
scoped_ptr<SourcePageHandle> handle) const override;
private:
@@ -53,24 +53,23 @@ class DistillerPageWebContents : public DistillerPage,
content::BrowserContext* browser_context,
const gfx::Size& render_view_size,
scoped_ptr<SourcePageHandleWebContents> optional_web_contents_handle);
- virtual ~DistillerPageWebContents();
+ ~DistillerPageWebContents() override;
// content::WebContentsDelegate implementation.
- virtual gfx::Size GetSizeForNewRenderView(
+ gfx::Size GetSizeForNewRenderView(
content::WebContents* web_contents) const override;
// content::WebContentsObserver implementation.
- virtual void DocumentLoadedInFrame(
+ void DocumentLoadedInFrame(
content::RenderFrameHost* render_frame_host) override;
- virtual void DidFailLoad(content::RenderFrameHost* render_frame_host,
- const GURL& validated_url,
- int error_code,
- const base::string16& error_description) override;
+ void DidFailLoad(content::RenderFrameHost* render_frame_host,
+ const GURL& validated_url,
+ int error_code,
+ const base::string16& error_description) override;
protected:
- virtual void DistillPageImpl(const GURL& url,
- const std::string& script) override;
+ void DistillPageImpl(const GURL& url, const std::string& script) override;
private:
friend class TestDistillerPageWebContents;

Powered by Google App Engine
This is Rietveld 408576698