Print preview page selection should not require a rerendering of draft pages.

chrome/renderer/print_web_view_helper.h

Index: chrome/renderer/print_web_view_helper.h
diff --git a/chrome/renderer/print_web_view_helper.h b/chrome/renderer/print_web_view_helper.h
index 4606c3716217f3d8db447e0272f6be95999e3217..71b284737d4e45d0e5ec5c3dd20d7cd292c08aa9 100644
--- a/chrome/renderer/print_web_view_helper.h
+++ b/chrome/renderer/print_web_view_helper.h
@@ -6,7 +6,6 @@
 #define CHROME_RENDERER_PRINT_WEB_VIEW_HELPER_H_
 #pragma once
 
-#include <utility>
 #include <vector>
 
 #include "base/memory/scoped_ptr.h"
@@ -135,10 +134,8 @@ class PrintWebViewHelper : public RenderViewObserver,
   // Initialize the print preview document.
   bool CreatePreviewDocument();
 
-  // Continue generating the print preview. |requested_preview_page_index|
-  // specifies the browser requested preview page index. It is 1-based or
-  // |printing::INVALID_PAGE_INDEX| to continue with next page.
-  void OnContinuePreview(int requested_preview_page_index);
+  // Continue generating the print preview.
+  void OnContinuePreview();
   // Renders a print preview page. |page_number| is 0-based.
   void RenderPreviewPage(int page_number);
   // Finalize the print preview document.
@@ -292,6 +289,7 @@ class PrintWebViewHelper : public RenderViewObserver,
 
   scoped_ptr<PrintMsg_PrintPages_Params> print_pages_params_;
   bool is_preview_;
+  bool is_complete_metafile_already_sent_;

[vandebo (ex-Chrome), 2011/08/04 21:45:02]

nit (if you like): already isn't really necessary.

[kmadhusu, 2011/08/05 22:55:22]

Done.

 
   // Used for scripted initiated printing blocking.
   base::Time last_cancelled_script_print_;
@@ -325,6 +323,9 @@ class PrintWebViewHelper : public RenderViewObserver,
     // rendering took.
     void RenderedPreviewPage(const base::TimeDelta& page_time);
 
+    // Updates the print preview context when the required pages are rendered.
+    void PreviewPagesRendered();
+
     // Finalizes the print preview document.
     void FinalizePreviewDocument();
 
@@ -339,11 +340,16 @@ class PrintWebViewHelper : public RenderViewObserver,
     bool IsReadyToRender() const;
     bool IsBusy() const;
     bool IsModifiable() const;
+    bool IsLastPageOfCompleteMetafile() const;
+
+    // Setters
+    void set_generate_draft_pages(bool generate_draft_pages);
 
     // Getters
     WebKit::WebFrame* frame() const;
     WebKit::WebNode* node() const;
     int total_page_count() const;
+    bool generate_draft_pages();
     printing::Metafile* metafile() const;
     const PrintMsg_Print_Params& print_params() const;
     const gfx::Size& GetPrintCanvasSize() const;
@@ -370,16 +376,17 @@ class PrintWebViewHelper : public RenderViewObserver,
     // Total page count in the renderer.
     int total_page_count_;
 
-    // Number of pages to render.
-    int actual_page_count_;
+    // List of page indices that needs to be rendered.

[vandebo (ex-Chrome), 2011/08/04 21:45:02]

nit: needs -> need

[kmadhusu, 2011/08/05 22:55:22]

Done.

+    std::vector<int> render_page_list_;
+
+    // Specifies the current list index.
+    int render_page_list_index_;
 
-    // The current page to render.
-    int current_page_number_;
+    // True, when draft pages needs to be generated.
+    bool generate_draft_pages_;
 
-    // |rendered_pages_| tracks which pages need to be printed as well as
-    // the page slot it should be printed in. See GetPageSlotForPage.
-    typedef std::pair<bool, int> PreviewPageInfo;
-    std::vector<PreviewPageInfo> rendered_pages_;
+    // Specifies the total number of pages in complete metafile.

[vandebo (ex-Chrome), 2011/08/04 21:45:02]

nit: in the complete

[kmadhusu, 2011/08/05 22:55:22]

Done.

+    int complete_metafile_page_count_;
 
     base::TimeDelta document_render_time_;
     base::TimeTicks begin_time_;