Print Preview: Fixing behavior of event listeners.

chrome/renderer/mock_printer.cc

Index: chrome/renderer/mock_printer.cc
diff --git a/chrome/renderer/mock_printer.cc b/chrome/renderer/mock_printer.cc
index 6fa65ccf408f0a309e962ad92e10abb3d35b6612..38da470fffa2bf2e2e5b8aa5a62fc3fc39330a96 100644
--- a/chrome/renderer/mock_printer.cc
+++ b/chrome/renderer/mock_printer.cc
@@ -35,7 +35,9 @@ MockPrinter::MockPrinter()
     current_document_cookie_(0),
     printer_status_(PRINTER_READY),
     number_pages_(0),
-    page_number_(0) {
+    page_number_(0),
+    is_first_request_(false),

[Lei Zhang, 2011/08/12 02:11:34]

Why don't we just default to true and 0 and be done with it? I think that should
work; correct me if I'm wrong.

[dpapad, 2011/08/12 19:01:28]

Done.

+    preview_request_id_(-1) {
   page_size_.SetSize(static_cast<int>(8.5 * dpi_),
                      static_cast<int>(11.0 * dpi_));
   printable_size_.SetSize(static_cast<int>((7.5 * dpi_)),
@@ -90,6 +92,8 @@ void MockPrinter::ScriptedPrint(int cookie,
   settings->params.document_cookie = document_cookie_;
   settings->params.page_size = page_size_;
   settings->params.printable_size = printable_size_;
+  settings->params.is_first_request = is_first_request_;
+  settings->params.preview_request_id = preview_request_id_;
   printer_status_ = PRINTER_PRINTING;
 }
 
@@ -99,6 +103,8 @@ void MockPrinter::UpdateSettings(int cookie,
 
   memset(params, 0, sizeof(PrintMsg_PrintPages_Params));
   SetPrintParams(&(params->params));
+  params->params.is_first_request = true;
+  params->params.preview_request_id = 0;
   printer_status_ = PRINTER_PRINTING;
 }
 
@@ -230,4 +236,6 @@ void MockPrinter::SetPrintParams(PrintMsg_Print_Params* params) {
   params->printable_size = printable_size_;
   params->margin_left = margin_left_;
   params->margin_top = margin_top_;
+  params->is_first_request = is_first_request_;
+  params->preview_request_id = preview_request_id_;
 }