Revert 96567 - Reland 96406 - Print preview page selection should not require a rerendering of dr...

chrome/renderer/print_web_view_helper_mac.mm

Index: chrome/renderer/print_web_view_helper_mac.mm
===================================================================
--- chrome/renderer/print_web_view_helper_mac.mm	(revision 96579)
+++ chrome/renderer/print_web_view_helper_mac.mm	(working copy)
@@ -66,30 +66,19 @@
                          printParams.printable_size.width(),
                          printParams.printable_size.height());
 
-  scoped_ptr<printing::Metafile> draft_metafile;
   printing::Metafile* initial_render_metafile =
       print_preview_context_.metafile();
-
-#if defined(USE_SKIA)
-  bool need_draft_metafile = print_preview_context_.IsModifiable() &&
-                             is_print_ready_metafile_sent_;
-#else
-  bool need_draft_metafile = print_preview_context_.IsModifiable() &&
-                             print_preview_context_.generate_draft_pages();
-  // NOTE: If the rendering page need to be in draft metafile and print ready
-  // metafile, we should always render to the draft metafile first and then
-  // copy that into the print ready metafile because CG does not allow us to do
-  // it in the other order.
-#endif
-
-  if (need_draft_metafile) {
-    draft_metafile.reset(new printing::PreviewMetafile());
+  scoped_ptr<printing::PreviewMetafile> draft_metafile;
+#if !defined(USE_SKIA)
+  if (print_preview_context_.IsModifiable()) {
+    draft_metafile.reset(new printing::PreviewMetafile);
     if (!draft_metafile->Init()) {
       DidFinishPrinting(FAIL_PREVIEW);
       return;
     }
     initial_render_metafile = draft_metafile.get();
   }
+#endif
 
   base::TimeTicks begin_time = base::TimeTicks::Now();
   RenderPage(printParams.page_size, content_area, scale_factor, page_number,
@@ -97,44 +86,38 @@
   print_preview_context_.RenderedPreviewPage(
       base::TimeTicks::Now() - begin_time);
 
-  if (draft_metafile.get()) {
+  if (print_preview_context_.IsModifiable()) {
+#if defined(USE_SKIA)
+    DCHECK(!draft_metafile.get());
+    draft_metafile.reset(
+        print_preview_context_.metafile()->GetMetafileForCurrentPage());
+#else
     draft_metafile->FinishDocument();
-#if !defined(USE_SKIA)
-    if (!is_print_ready_metafile_sent_) {
-      // With CG, we rendered into a new metafile so we could get it as a draft
-      // document.  Now we need to add it to print ready document.  But the
-      // document has already been scaled and adjusted for margins, so do a 1:1
-      // drawing.
-      printing::Metafile* print_ready_metafile =
-          print_preview_context_.metafile();
-      bool success = print_ready_metafile->StartPage(
-          printParams.page_size, gfx::Rect(printParams.page_size), 1.0);
-      DCHECK(success);
-      // StartPage unconditionally flips the content over, flip it back since it
-      // was already flipped in |draft_metafile|.
-      CGContextTranslateCTM(print_ready_metafile->context(), 0,
-                            printParams.page_size.height());
-      CGContextScaleCTM(print_ready_metafile->context(), 1.0, -1.0);
-      draft_metafile->RenderPage(1,
-                                 print_ready_metafile->context(),
-                                 draft_metafile->GetPageBounds(1).ToCGRect(),
-                                 false /* shrink_to_fit */,
-                                 false /* stretch_to_fit */,
-                                 true /* center_horizontally */,
-                                 true /* center_vertically */);
-      print_ready_metafile->FinishPage();
-    }
+
+    // With CG, we rendered into a new metafile so we could get it as a draft
+    // document.  Now we need to add it to complete document.  But the document
+    // has already been scaled and adjusted for margins, so do a 1:1 drawing.
+    printing::Metafile* complete_metafile = print_preview_context_.metafile();
+    bool success = complete_metafile->StartPage(
+        printParams.page_size, gfx::Rect(printParams.page_size), 1.0);
+    DCHECK(success);
+    // StartPage unconditionally flips the content over, flip it back since it
+    // was already flipped in |draft_metafile|.
+    CGContextTranslateCTM(complete_metafile->context(), 0,
+                          printParams.page_size.height());
+    CGContextScaleCTM(complete_metafile->context(), 1.0, -1.0);
+
+    draft_metafile->RenderPage(1,
+                               complete_metafile->context(),
+                               draft_metafile->GetPageBounds(1).ToCGRect(),
+                               false /* shrink_to_fit */,
+                               false /* stretch_to_fit */,
+                               true  /* center_horizontally */,
+                               true  /* center_vertically */);
+    complete_metafile->FinishPage();
 #endif
-  } else {
-#if defined(USE_SKIA)
-    if (print_preview_context_.IsModifiable() &&
-        print_preview_context_.generate_draft_pages()) {
-      DCHECK(!draft_metafile.get());
-      draft_metafile.reset(
-          print_preview_context_.metafile()->GetMetafileForCurrentPage());
-    }
-#endif
   }
+
   PreviewPageRendered(page_number, draft_metafile.get());
 }
 
@@ -154,8 +137,6 @@
     canvas->unref();  // SkRefPtr and new both took a reference.
     WebKit::WebCanvas* canvasPtr = canvas.get();
     printing::MetafileSkiaWrapper::SetMetafileOnCanvas(canvasPtr, metafile);
-    printing::MetafileSkiaWrapper::SetDraftMode(canvasPtr,
-                                                is_print_ready_metafile_sent_);
 #else
     bool success = metafile->StartPage(page_size, content_area, scale_factor);
     DCHECK(success);