Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4208)

Unified Diff: chrome/renderer/mock_render_thread.cc

Issue 7550022: Print Preview: Fixing behavior of event listeners. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressing comments Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/mock_render_thread.cc
diff --git a/chrome/renderer/mock_render_thread.cc b/chrome/renderer/mock_render_thread.cc
index aac71561a37d6ce2e0e2e940dcf405481eae0cfe..b44355955b62839cba36ec41e08e0e096bd9129b 100644
--- a/chrome/renderer/mock_render_thread.cc
+++ b/chrome/renderer/mock_render_thread.cc
@@ -211,10 +211,9 @@ void MockRenderThread::OnDidPrintPage(
printer_->PrintPage(params);
}
-void MockRenderThread::OnDidGetPreviewPageCount(int document_cookie,
- int number_pages,
- bool is_modifiable) {
- print_preview_pages_remaining_ = number_pages;
+void MockRenderThread::OnDidGetPreviewPageCount(
+ const PrintHostMsg_DidGetPreviewPageCount_Params& params) {
+ print_preview_pages_remaining_ = params.page_count;
}
void MockRenderThread::OnDidPreviewPage(
@@ -235,9 +234,11 @@ void MockRenderThread::OnUpdatePrintSettings(
!job_settings.GetBoolean(printing::kSettingCollate, NULL) ||
!job_settings.GetBoolean(printing::kSettingColor, NULL) ||
!job_settings.GetBoolean(printing::kSettingPrintToPDF, NULL) ||
+ !job_settings.GetBoolean(printing::kIsFirstRequest, NULL) ||
!job_settings.GetString(printing::kSettingDeviceName, &dummy_string) ||
!job_settings.GetInteger(printing::kSettingDuplexMode, NULL) ||
- !job_settings.GetInteger(printing::kSettingCopies, NULL)) {
+ !job_settings.GetInteger(printing::kSettingCopies, NULL) ||
+ !job_settings.GetInteger(printing::kPreviewRequestID, NULL)) {
return;
}

Powered by Google App Engine
This is Rietveld 408576698