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

Unified Diff: chrome/renderer/render_view.h

Issue 125082: Add Print Selection support to Chrome. This change is fairly involved since ... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 6 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 | « chrome/renderer/print_web_view_helper.cc ('k') | chrome/renderer/render_view.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/renderer/render_view.h
===================================================================
--- chrome/renderer/render_view.h (revision 18804)
+++ chrome/renderer/render_view.h (working copy)
@@ -461,6 +461,7 @@
// RenderView IPC message handlers
void SendThumbnail();
void OnPrintPages();
+ void OnPrintingDone(int document_cookie, bool success);
void OnNavigate(const ViewMsg_Navigate_Params& params);
void OnStop();
void OnLoadAlternateHTMLText(const std::string& html_contents,
@@ -793,8 +794,9 @@
// to the new navigation is created. See DidCreateDataSource.
scoped_ptr<NavigationState> pending_navigation_state_;
- // Need for printing
- scoped_ptr<PrintWebViewHelper> print_render_view_;
+ // PrintWebViewHelper handles printing. Note that this object is constructed
+ // when printing for the first time but only destroyed with the RenderView.
+ scoped_ptr<PrintWebViewHelper> print_helper_;
RendererPreferences renderer_preferences_;
« no previous file with comments | « chrome/renderer/print_web_view_helper.cc ('k') | chrome/renderer/render_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698