Chromium Code Reviews| Index: components/pdf/browser/pdf_web_contents_helper.h |
| diff --git a/components/pdf/browser/pdf_web_contents_helper.h b/components/pdf/browser/pdf_web_contents_helper.h |
| index c9a2e5897d08e94dc2aef43b6e6e9de331811404..81f51198a51f0059f18752e5e2ba2bf972e7a602 100644 |
| --- a/components/pdf/browser/pdf_web_contents_helper.h |
| +++ b/components/pdf/browser/pdf_web_contents_helper.h |
| @@ -10,9 +10,10 @@ |
| #include "base/callback.h" |
| #include "base/macros.h" |
| +#include "components/pdf/common/pdf.mojom.h" |
| +#include "content/public/browser/web_contents_binding_set.h" |
| #include "content/public/browser/web_contents_observer.h" |
| #include "content/public/browser/web_contents_user_data.h" |
| -#include "ipc/ipc_message.h" |
| namespace content { |
| class WebContents; |
| @@ -25,7 +26,8 @@ class PDFWebContentsHelperClient; |
| // Per-WebContents class to handle PDF messages. |
| class PDFWebContentsHelper |
| : public content::WebContentsObserver, |
| - public content::WebContentsUserData<PDFWebContentsHelper> { |
| + public content::WebContentsUserData<PDFWebContentsHelper>, |
| + public mojom::PdfService { |
| public: |
| static void CreateForWebContentsWithClient( |
| content::WebContents* contents, |
| @@ -36,14 +38,12 @@ class PDFWebContentsHelper |
| std::unique_ptr<PDFWebContentsHelperClient> client); |
| ~PDFWebContentsHelper() override; |
| - // content::WebContentsObserver overrides: |
| - bool OnMessageReceived(const IPC::Message& message, |
| - content::RenderFrameHost* render_frame_host) override; |
| + // mojom::PdfService: |
| + void HasUnsupportedFeature() override; |
| + void SaveUrlAs(const GURL& url, const content::Referrer& referrer) override; |
| + void UpdateContentRestrictions(int32_t content_restrictions) override; |
| - // Message handlers. |
| - void OnHasUnsupportedFeature(); |
| - void OnSaveURLAs(const GURL& url, const content::Referrer& referrer); |
| - void OnUpdateContentRestrictions(int content_restrictions); |
| + content::WebContentsFrameBindingSet<mojom::PdfService> pdf_service_bindings_; |
|
dsinclair
2017/04/03 13:23:01
nit: no blank link
benwells
2017/04/04 01:38:18
Done.
|
| std::unique_ptr<PDFWebContentsHelperClient> client_; |