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

Unified Diff: chrome/browser/printing/print_preview_message_handler.cc

Issue 2962983002: Print Preview: change getPreview to cr.sendWithPromise (Closed)
Patch Set: Rebase Created 3 years, 6 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/browser/printing/print_preview_message_handler.cc
diff --git a/chrome/browser/printing/print_preview_message_handler.cc b/chrome/browser/printing/print_preview_message_handler.cc
index 89cc4977ce55084e3bcb7b912c5ce9b230723f15..ac67415db7061b7674ed5fb895337927ba90132f 100644
--- a/chrome/browser/printing/print_preview_message_handler.cc
+++ b/chrome/browser/printing/print_preview_message_handler.cc
@@ -166,13 +166,14 @@ void PrintPreviewMessageHandler::OnMetafileReadyForPrinting(
params.expected_pages_count, params.preview_request_id);
}
-void PrintPreviewMessageHandler::OnPrintPreviewFailed(int document_cookie) {
+void PrintPreviewMessageHandler::OnPrintPreviewFailed(int document_cookie,
+ int request_id) {
StopWorker(document_cookie);
PrintPreviewUI* print_preview_ui = GetPrintPreviewUI();
if (!print_preview_ui)
return;
- print_preview_ui->OnPrintPreviewFailed();
+ print_preview_ui->OnPrintPreviewFailed(request_id);
}
void PrintPreviewMessageHandler::OnDidGetDefaultPageLayout(
@@ -192,12 +193,13 @@ void PrintPreviewMessageHandler::OnPrintPreviewCancelled(int document_cookie) {
StopWorker(document_cookie);
}
-void PrintPreviewMessageHandler::OnInvalidPrinterSettings(int document_cookie) {
+void PrintPreviewMessageHandler::OnInvalidPrinterSettings(int document_cookie,
+ int request_id) {
StopWorker(document_cookie);
PrintPreviewUI* print_preview_ui = GetPrintPreviewUI();
if (!print_preview_ui)
return;
- print_preview_ui->OnInvalidPrinterSettings();
+ print_preview_ui->OnInvalidPrinterSettings(request_id);
}
void PrintPreviewMessageHandler::OnSetOptionsFromDocument(

Powered by Google App Engine
This is Rietveld 408576698