Print Preview: Fixing behavior of event listeners.

chrome/renderer/mock_render_thread.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/renderer/mock_render_thread.h"
 
 #include <fcntl.h>
 
 #include "base/file_util.h"
 #include "base/process_util.h"
@@ skipping 195 matching lines @@
 }
 
 void MockRenderThread::OnDidPrintPage(
     const PrintHostMsg_DidPrintPage_Params& params) {
   if (printer_.get())
     printer_->PrintPage(params);
 }
 
 void MockRenderThread::OnDidGetPreviewPageCount(int document_cookie,
                                                 int number_pages,
-                                                bool is_modifiable) {
+                                                bool is_modifiable,
+                                                int preview_request_id) {
   print_preview_pages_remaining_ = number_pages;
 }
 
 void MockRenderThread::OnDidPreviewPage(
     const PrintHostMsg_DidPreviewPage_Params& params) {
   if (params.page_number < printing::FIRST_PAGE_INDEX)
     return;
   print_preview_pages_remaining_--;
 }
 
 void MockRenderThread::OnUpdatePrintSettings(
     int document_cookie,
     const DictionaryValue& job_settings,
     PrintMsg_PrintPages_Params* params) {
   // Check and make sure the required settings are all there.
   // We don't actually care about the values.
   std::string dummy_string;
   if (!job_settings.GetBoolean(printing::kSettingLandscape, NULL) ||
       !job_settings.GetBoolean(printing::kSettingCollate, NULL) ||
       !job_settings.GetBoolean(printing::kSettingColor, NULL) ||
       !job_settings.GetBoolean(printing::kSettingPrintToPDF, NULL) ||
       !job_settings.GetString(printing::kSettingDeviceName, &dummy_string) ||
       !job_settings.GetInteger(printing::kSettingDuplexMode, NULL) ||
       !job_settings.GetInteger(printing::kSettingCopies, NULL)) {

[kmadhusu, 2011/08/08 22:20:45]

Make sure job_settings has "IsFirstRequest" and "requestID" keys.

[dpapad, 2011/08/08 23:41:43]

Done.

     return;
   }
 
   // Just return the default settings.
   if (printer_.get())
     printer_->UpdateSettings(document_cookie, params);
 }
 
 void MockRenderThread::set_print_dialog_user_response(bool response) {
   print_dialog_user_response_ = response;
 }
 
 int MockRenderThread::print_preview_pages_remaining() {
   return print_preview_pages_remaining_;
 }