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

Unified Diff: chrome/test/data/webui/print_preview/native_layer_stub.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/test/data/webui/print_preview/native_layer_stub.js
diff --git a/chrome/test/data/webui/print_preview/native_layer_stub.js b/chrome/test/data/webui/print_preview/native_layer_stub.js
index 495d22458ad9b86ca49de5b33263ef04addb4e81..e95b7081b86f6304ad6f47ece2d35f4d981019c4 100644
--- a/chrome/test/data/webui/print_preview/native_layer_stub.js
+++ b/chrome/test/data/webui/print_preview/native_layer_stub.js
@@ -13,6 +13,7 @@ cr.define('print_preview', function() {
'getInitialSettings',
'getPrinters',
'getExtensionPrinters',
+ 'getPreview',
'getPrivetPrinters',
'getPrinterCapabilities',
'print',
@@ -25,12 +26,6 @@ cr.define('print_preview', function() {
*/
this.eventTarget_ = new cr.EventTarget();
- /**
- * @private {boolean} Whether the native layer has set the generate draft
- * parameter when requesting an updated preview.
- */
- this.generateDraft_ = false;
-
/**
* @private {!print_preview.NativeInitialSettings} The initial settings
* to be used for the response to a |getInitialSettings| call.
@@ -62,6 +57,11 @@ cr.define('print_preview', function() {
* @private {boolean} Whether the printer setup request should be rejected.
*/
this.shouldRejectPrinterSetup_ = false;
+
+ /**
+ * @private {string} The ID of a printer with a bad driver.
+ */
+ this.badPrinterId_ = '';
}
NativeLayerStub.prototype = {
@@ -85,6 +85,24 @@ cr.define('print_preview', function() {
return Promise.resolve(true);
},
+ /** @override */
+ getPreview: function(
+ destination, printTicketStore, documentInfo, generateDraft, requestId) {
+ this.methodCalled('getPreview', {
+ destination: destination,
+ printTicketStore: printTicketStore,
+ documentInfo: documentInfo,
+ generateDraft: generateDraft,
+ requestId: requestId,
+ });
+ var rejectString = print_preview.PreviewArea.EventType.SETTINGS_INVALID;
+ rejectString = rejectString.substring(
+ rejectString.lastIndexOf(".") + 1, rejectString.length);
+ return destination.id == this.badPrinterId_ ?
+ Promise.reject(rejectString) :
+ Promise.resolve(requestId);
+ },
+
/** @override */
getPrivetPrinters: function() {
this.methodCalled('getPrivetPrinters');
@@ -119,11 +137,6 @@ cr.define('print_preview', function() {
/** Stubs for |print_preview.NativeLayer| methods that call C++ handlers. */
previewReadyForTest: function() {},
-
- startGetPreview: function(destination, printTicketStore, documentInfo,
- generateDraft, requestId) {
- this.generateDraft_ = generateDraft;
- },
startHideDialog: function () {},
/** @return {!cr.EventTarget} The native layer event target. */
@@ -134,9 +147,6 @@ cr.define('print_preview', function() {
this.eventTarget_ = eventTarget;
},
- /** @return {boolean} Whether a new draft was requested for preview. */
- generateDraft: function() { return this.generateDraft_; },
-
/**
* @param {!print_preview.NativeInitialSettings} settings The settings
* to return as a response to |getInitialSettings|.
@@ -174,6 +184,15 @@ cr.define('print_preview', function() {
this.shouldRejectPrinterSetup_ = reject;
this.setupPrinterResponse_ = response;
},
+
+ /**
+ * @param {string} bad_id The printer ID that should cause an
+ * SETTINGS_INVALID error in response to a preview request. Models a
+ * bad printer driver.
+ */
+ setInvalidPrinterId: function(id) {
+ this.badPrinterId_ = id;
+ },
};
return {
« no previous file with comments | « chrome/browser/ui/webui/print_preview/print_preview_ui.cc ('k') | chrome/test/data/webui/print_preview/print_preview_tests.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698