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

Unified Diff: chrome/browser/resources/print_preview/native_layer.js

Issue 2962983002: Print Preview: change getPreview to cr.sendWithPromise (Closed)
Patch Set: Fix check 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/resources/print_preview/native_layer.js
diff --git a/chrome/browser/resources/print_preview/native_layer.js b/chrome/browser/resources/print_preview/native_layer.js
index 7e45ed32c5c7f7466f6de74763ec4c37b059d487..d0b37df86f231f5aeb2b2233d1a9e44e6a217975 100644
--- a/chrome/browser/resources/print_preview/native_layer.js
+++ b/chrome/browser/resources/print_preview/native_layer.js
@@ -87,13 +87,10 @@ cr.define('print_preview', function() {
function NativeLayer() {
// Bind global handlers
global.reloadPrintersList = this.onReloadPrintersList_.bind(this);
- global.printPreviewFailed = this.onPrintPreviewFailed_.bind(this);
- global.invalidPrinterSettings = this.onInvalidPrinterSettings_.bind(this);
global.onDidGetDefaultPageLayout =
this.onDidGetDefaultPageLayout_.bind(this);
global.onDidGetPreviewPageCount = this.onDidGetPreviewPageCount_.bind(this);
global.onDidPreviewPage = this.onDidPreviewPage_.bind(this);
- global.updatePrintPreview = this.onUpdatePrintPreview_.bind(this);
global.onEnableManipulateSettingsForTest =
this.onEnableManipulateSettingsForTest_.bind(this);
global.printPresetOptionsFromDocument =
@@ -330,8 +327,10 @@ cr.define('print_preview', function() {
* @param {!print_preview.DocumentInfo} documentInfo Document data model.
* @param {boolean} generateDraft Tell the renderer to re-render.
* @param {number} requestId ID of the preview request.
+ * @return {!Promise<number>} Promise that resolves with the unique ID of
+ * the preview UI when the preview has been generated.
*/
- startGetPreview: function(
+ getPreview: function(
destination, printTicketStore, documentInfo, generateDraft, requestId) {
assert(
printTicketStore.isTicketValidForPreview(),
@@ -394,9 +393,9 @@ cr.define('print_preview', function() {
};
}
- chrome.send('getPreview', [
- JSON.stringify(ticket), requestId > 0 ? documentInfo.pageCount : -1
- ]);
+ return cr.sendWithPromise(
+ 'getPreview', JSON.stringify(ticket),
+ requestId > 0 ? documentInfo.pageCount : -1);
},
/**
@@ -558,26 +557,6 @@ cr.define('print_preview', function() {
this.eventTarget_, NativeLayer.EventType.DESTINATIONS_RELOAD);
},
- /**
- * Display an error message when print preview fails.
- * Called from PrintPreviewMessageHandler::OnPrintPreviewFailed().
- * @private
- */
- onPrintPreviewFailed_: function() {
- cr.dispatchSimpleEvent(
- this.eventTarget_, NativeLayer.EventType.PREVIEW_GENERATION_FAIL);
- },
-
- /**
- * Display an error message when encountered invalid printer settings.
- * Called from PrintPreviewMessageHandler::OnInvalidPrinterSettings().
- * @private
- */
- onInvalidPrinterSettings_: function() {
- cr.dispatchSimpleEvent(
- this.eventTarget_, NativeLayer.EventType.SETTINGS_INVALID);
- },
-
/**
* @param {{contentWidth: number, contentHeight: number, marginLeft: number,
* marginRight: number, marginTop: number, marginBottom: number,
@@ -635,23 +614,6 @@ cr.define('print_preview', function() {
this.eventTarget_.dispatchEvent(pagePreviewGenEvent);
},
- /**
- * Update the print preview when new preview data is available.
- * Create the PDF plugin as needed.
- * Called from PrintPreviewUI::PreviewDataIsAvailable().
- * @param {number} previewUid Preview unique identifier.
- * @param {number} previewResponseId The preview request id that resulted in
- * this response.
- * @private
- */
- onUpdatePrintPreview_: function(previewUid, previewResponseId) {
- var previewGenDoneEvent =
- new Event(NativeLayer.EventType.PREVIEW_GENERATION_DONE);
- previewGenDoneEvent.previewUid = previewUid;
- previewGenDoneEvent.previewResponseId = previewResponseId;
- this.eventTarget_.dispatchEvent(previewGenDoneEvent);
- },
-
/**
* Updates print preset options from source PDF document.
* Called from PrintPreviewUI::OnSetOptionsFromDocument().
« no previous file with comments | « chrome/browser/printing/print_preview_message_handler.cc ('k') | chrome/browser/resources/print_preview/preview_generator.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698