Move Start/EndPage() from NewPage/PageDone() to inside the metafile.

printing/printing_context_win.cc

Index: printing/printing_context_win.cc
diff --git a/printing/printing_context_win.cc b/printing/printing_context_win.cc
index 467bf3c7ad52b3c00d4349d98839e82cca5ac298..eb10947a81235d7b65a8785bf8139d55e2f6deb1 100644
--- a/printing/printing_context_win.cc
+++ b/printing/printing_context_win.cc
@@ -301,21 +301,14 @@ PrintingContext::Result PrintingContextWin::NewPage() {
 
   DCHECK(context_);
   DCHECK(in_print_job_);
-
-  // Inform the driver that the application is about to begin sending data.

[vandebo (ex-Chrome), 2011/03/15 17:23:06]

Add a comment similar to the one in the Linux version of this method.

[dpapad, 2011/03/15 20:32:32]

Done.

-  if (StartPage(context_) <= 0)
-    return OnError();
-
   return OK;
 }
 
 PrintingContext::Result PrintingContextWin::PageDone() {
   if (abort_printing_)
     return CANCEL;
-  DCHECK(in_print_job_);
 
-  if (EndPage(context_) <= 0)
-    return OnError();
+  DCHECK(in_print_job_);
   return OK;
 }