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..5da429fb19c9dcae7ce52e5888fcae201390658e 100644
--- a/printing/printing_context_win.cc
+++ b/printing/printing_context_win.cc
@@ -298,13 +298,10 @@ PrintingContext::Result PrintingContextWin::NewDocument(
 PrintingContext::Result PrintingContextWin::NewPage() {
   if (abort_printing_)
     return CANCEL;
-
   DCHECK(context_);
   DCHECK(in_print_job_);
 
-  // Inform the driver that the application is about to begin sending data.
-  if (StartPage(context_) <= 0)
-    return OnError();
+  // Intentional No-op.

[vandebo (ex-Chrome), 2011/03/15 20:41:39]

Hmm, maybe we should note that NativeMetafile::SafePlayback takes care of
calling ::StartPage()

[dpapad, 2011/03/15 20:56:44]

Done.

 
   return OK;
 }
@@ -314,8 +311,8 @@ PrintingContext::Result PrintingContextWin::PageDone() {
     return CANCEL;
   DCHECK(in_print_job_);
 
-  if (EndPage(context_) <= 0)
-    return OnError();
+  // Intentional No-op.
+
   return OK;
 }