Refactor print functionality in RenderView. Moved most of the print functionality to a new class that will later be the base for asynchronous printing support.

This is to make further changes to printing easier.  There is no functional change and the code continues to be tested through the RenderView tests.


Review URL: http://codereview.chromium.org/99368

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17491 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 9 deletions

diff --git a/chrome/renderer/render_view.h b/chrome/renderer/render_view.h
index 0702fbe..bd216ae 100644
--- a/chrome/renderer/render_view.h
+++ b/chrome/renderer/render_view.h
@@ -46,6 +46,7 @@ class DebugMessageHandler;
 class DevToolsAgent;
 class DevToolsClient;
 class FilePath;
+class PrintWebViewHelper;
 class GURL;
 class RenderThread;
 class ResourceDispatcher;
@@ -58,7 +59,6 @@ struct FindInPageRequest;
 struct ThumbnailScore;
 struct ViewMsg_Navigate_Params;
 struct ViewMsg_PrintPage_Params;
-struct ViewMsg_PrintPages_Params;
 struct ViewMsg_Print_Params;
 struct ViewMsg_UploadFile_Params;
 struct WebDropData;
@@ -588,14 +588,6 @@ class RenderView : public RenderWidget,
   // a custom background.
   void OnSetBackground(const SkBitmap& background);
 
-  // Prints the page listed in |params|.
-  void PrintPage(const ViewMsg_PrintPage_Params& params,
-                 const gfx::Size& canvas_size,
-                 WebFrame* frame);
-
-  // Prints all the pages listed in |params|.
-  void PrintPages(const ViewMsg_PrintPages_Params& params, WebFrame* frame);
-
   // Attempt to upload the file that we are trying to process if any.
   // Reset the pending file upload data if the form was successfully
   // posted.
@@ -812,6 +804,9 @@ class RenderView : public RenderWidget,
   class NavigationState;
   scoped_ptr<NavigationState> pending_navigation_state_;
 
+  // Need for printing
+  scoped_ptr<PrintWebViewHelper> print_render_view_;
+
   DISALLOW_COPY_AND_ASSIGN(RenderView);
 };