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

Side by Side 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef COMPONENTS_DOM_DISTILLER_CONTENT_DISTILLER_PAGE_WEB_CONTENTS_H_ 5 #ifndef COMPONENTS_DOM_DISTILLER_CONTENT_DISTILLER_PAGE_WEB_CONTENTS_H_
6 #define COMPONENTS_DOM_DISTILLER_CONTENT_DISTILLER_PAGE_WEB_CONTENTS_H_ 6 #define COMPONENTS_DOM_DISTILLER_CONTENT_DISTILLER_PAGE_WEB_CONTENTS_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
11 #include "components/dom_distiller/core/distiller_page.h" 11 #include "components/dom_distiller/core/distiller_page.h"
12 #include "content/public/browser/web_contents.h" 12 #include "content/public/browser/web_contents.h"
13 #include "content/public/browser/web_contents_delegate.h" 13 #include "content/public/browser/web_contents_delegate.h"
14 #include "content/public/browser/web_contents_observer.h" 14 #include "content/public/browser/web_contents_observer.h"
15 #include "url/gurl.h" 15 #include "url/gurl.h"
16 16
17 namespace dom_distiller { 17 namespace dom_distiller {
18 18
19 class SourcePageHandleWebContents : public SourcePageHandle { 19 class SourcePageHandleWebContents : public SourcePageHandle {
20 public: 20 public:
21 explicit SourcePageHandleWebContents( 21 explicit SourcePageHandleWebContents(
22 scoped_ptr<content::WebContents> web_contents); 22 scoped_ptr<content::WebContents> web_contents);
23 virtual ~SourcePageHandleWebContents(); 23 ~SourcePageHandleWebContents() override;
24 24
25 scoped_ptr<content::WebContents> GetWebContents(); 25 scoped_ptr<content::WebContents> GetWebContents();
26 26
27 private: 27 private:
28 // The WebContents this class owns. 28 // The WebContents this class owns.
29 scoped_ptr<content::WebContents> web_contents_; 29 scoped_ptr<content::WebContents> web_contents_;
30 }; 30 };
31 31
32 class DistillerPageWebContentsFactory : public DistillerPageFactory { 32 class DistillerPageWebContentsFactory : public DistillerPageFactory {
33 public: 33 public:
34 explicit DistillerPageWebContentsFactory( 34 explicit DistillerPageWebContentsFactory(
35 content::BrowserContext* browser_context) 35 content::BrowserContext* browser_context)
36 : DistillerPageFactory(), browser_context_(browser_context) {} 36 : DistillerPageFactory(), browser_context_(browser_context) {}
37 virtual ~DistillerPageWebContentsFactory() {} 37 ~DistillerPageWebContentsFactory() override {}
38 38
39 virtual scoped_ptr<DistillerPage> CreateDistillerPage( 39 scoped_ptr<DistillerPage> CreateDistillerPage(
40 const gfx::Size& render_view_size) const override; 40 const gfx::Size& render_view_size) const override;
41 virtual scoped_ptr<DistillerPage> CreateDistillerPageWithHandle( 41 scoped_ptr<DistillerPage> CreateDistillerPageWithHandle(
42 scoped_ptr<SourcePageHandle> handle) const override; 42 scoped_ptr<SourcePageHandle> handle) const override;
43 43
44 private: 44 private:
45 content::BrowserContext* browser_context_; 45 content::BrowserContext* browser_context_;
46 }; 46 };
47 47
48 class DistillerPageWebContents : public DistillerPage, 48 class DistillerPageWebContents : public DistillerPage,
49 public content::WebContentsDelegate, 49 public content::WebContentsDelegate,
50 public content::WebContentsObserver { 50 public content::WebContentsObserver {
51 public: 51 public:
52 DistillerPageWebContents( 52 DistillerPageWebContents(
53 content::BrowserContext* browser_context, 53 content::BrowserContext* browser_context,
54 const gfx::Size& render_view_size, 54 const gfx::Size& render_view_size,
55 scoped_ptr<SourcePageHandleWebContents> optional_web_contents_handle); 55 scoped_ptr<SourcePageHandleWebContents> optional_web_contents_handle);
56 virtual ~DistillerPageWebContents(); 56 ~DistillerPageWebContents() override;
57 57
58 // content::WebContentsDelegate implementation. 58 // content::WebContentsDelegate implementation.
59 virtual gfx::Size GetSizeForNewRenderView( 59 gfx::Size GetSizeForNewRenderView(
60 content::WebContents* web_contents) const override; 60 content::WebContents* web_contents) const override;
61 61
62 // content::WebContentsObserver implementation. 62 // content::WebContentsObserver implementation.
63 virtual void DocumentLoadedInFrame( 63 void DocumentLoadedInFrame(
64 content::RenderFrameHost* render_frame_host) override; 64 content::RenderFrameHost* render_frame_host) override;
65 65
66 virtual void DidFailLoad(content::RenderFrameHost* render_frame_host, 66 void DidFailLoad(content::RenderFrameHost* render_frame_host,
67 const GURL& validated_url, 67 const GURL& validated_url,
68 int error_code, 68 int error_code,
69 const base::string16& error_description) override; 69 const base::string16& error_description) override;
70 70
71 protected: 71 protected:
72 virtual void DistillPageImpl(const GURL& url, 72 void DistillPageImpl(const GURL& url, const std::string& script) override;
73 const std::string& script) override;
74 73
75 private: 74 private:
76 friend class TestDistillerPageWebContents; 75 friend class TestDistillerPageWebContents;
77 76
78 enum State { 77 enum State {
79 // The page distiller is idle. 78 // The page distiller is idle.
80 IDLE, 79 IDLE,
81 // A page is currently loading. 80 // A page is currently loading.
82 LOADING_PAGE, 81 LOADING_PAGE,
83 // There was an error processing the page. 82 // There was an error processing the page.
(...skipping 23 matching lines...) Expand all
107 106
108 scoped_ptr<content::WebContents> web_contents_; 107 scoped_ptr<content::WebContents> web_contents_;
109 content::BrowserContext* browser_context_; 108 content::BrowserContext* browser_context_;
110 gfx::Size render_view_size_; 109 gfx::Size render_view_size_;
111 DISALLOW_COPY_AND_ASSIGN(DistillerPageWebContents); 110 DISALLOW_COPY_AND_ASSIGN(DistillerPageWebContents);
112 }; 111 };
113 112
114 } // namespace dom_distiller 113 } // namespace dom_distiller
115 114
116 #endif // COMPONENTS_DOM_DISTILLER_CONTENT_DISTILLER_PAGE_WEB_CONTENTS_H_ 115 #endif // COMPONENTS_DOM_DISTILLER_CONTENT_DISTILLER_PAGE_WEB_CONTENTS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698