Chromium Code Reviews| Index: chrome/renderer/printing/print_web_view_helper.h |
| diff --git a/chrome/renderer/printing/print_web_view_helper.h b/chrome/renderer/printing/print_web_view_helper.h |
| index 9ba081094ee0b3c506e37d4a080a2e831b8cb37a..50f8516c7218a36e7441772250f5f4294fd6826b 100644 |
| --- a/chrome/renderer/printing/print_web_view_helper.h |
| +++ b/chrome/renderer/printing/print_web_view_helper.h |
| @@ -66,7 +66,19 @@ class PrintWebViewHelper |
| : public content::RenderViewObserver, |
| public content::RenderViewObserverTracker<PrintWebViewHelper> { |
| public: |
| - explicit PrintWebViewHelper(content::RenderView* render_view); |
| + |
| + class Delegate { |
| + public: |
| + virtual ~Delegate() {} |
| + virtual bool CancelPrerender(content::RenderView* render_view, |
| + int routing_id) = 0; |
| + }; |
| + |
| + PrintWebViewHelper(content::RenderView* render_view, |
| + bool out_of_process_pdf_enabled, |
| + bool print_preview_disabled, |
| + const char* pdf_extension_id, |
|
Vitaly Buka (NO REVIEWS)
2015/01/06 19:29:24
please use const std::string&
dgn
2015/01/06 21:01:45
Done.
|
| + scoped_ptr<Delegate> delegate); |
| ~PrintWebViewHelper() override; |
| // Disable print preview and switch to system dialog printing even if full |
| @@ -327,6 +339,16 @@ class PrintWebViewHelper |
| // True, when printing from print preview. |
| bool print_for_preview_; |
| + // Whether the content to print could be nested in an iframe. |
| + const bool out_of_process_pdf_enabled_; |
| + |
| + // Used to test if the nested iframe holding the content to print is the |
| + // pdf extension. Only used when |ENABLE_EXTENSIONS| is defined. |
| + const char* pdf_extension_id_; |
|
Vitaly Buka (NO REVIEWS)
2015/01/06 19:29:24
use std::string here
you can't be sure that it's n
dgn
2015/01/06 21:01:45
Done.
|
| + |
| + // Used to check the prerendering status. |
| + const scoped_ptr<Delegate> delegate_; |
| + |
| // Keeps track of the state of print preview between messages. |
| // TODO(vitalybuka): Create PrintPreviewContext when needed and delete after |
| // use. Now it's interaction with various messages is confusing. |