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

Unified Diff: chrome/test/data/webui/print_preview/native_layer_stub.js

Issue 2969383003: Print Preview: Finish removing global Javascript functions. (Closed)
Patch Set: Fix test Created 3 years, 5 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 e95b7081b86f6304ad6f47ece2d35f4d981019c4..f3122684504d016cddea63a27edb04634eac765c 100644
--- a/chrome/test/data/webui/print_preview/native_layer_stub.js
+++ b/chrome/test/data/webui/print_preview/native_layer_stub.js
@@ -20,12 +20,6 @@ cr.define('print_preview', function() {
'setupPrinter'
]);
- /**
- * @private {!cr.EventTarget} The event target used for dispatching and
- * receiving events.
- */
- this.eventTarget_ = new cr.EventTarget();
-
/**
* @private {!print_preview.NativeInitialSettings} The initial settings
* to be used for the response to a |getInitialSettings| call.
@@ -95,12 +89,31 @@ cr.define('print_preview', function() {
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);
+ if (destination.id == this.badPrinterId_) {
+ var rejectString = print_preview.PreviewArea.EventType.SETTINGS_INVALID;
+ rejectString = rejectString.substring(
+ rejectString.lastIndexOf('.') + 1, rejectString.length);
+ return Promise.reject(rejectString);
+ }
+ var pageRanges = printTicketStore.pageRange.getDocumentPageRanges();
+ if (pageRanges.length == 0) { // assume full length document, 1 page.
+ cr.webUIListenerCallback('page-count-ready', 1, requestId, 100);
+ cr.webUIListenerCallback('page-preview-ready', 0, 0, requestId);
+ } else {
+ var pages = pageRanges.reduce(function(soFar, range) {
+ for (var page = range.from; page <= range.to; page++) {
+ soFar.push(page);
+ }
+ return soFar;
+ }, []);
+ cr.webUIListenerCallback(
+ 'page-count-ready', pages.length, requestId, 100);
+ pages.forEach(function(page) {
+ cr.webUIListenerCallback(
+ 'page-preview-ready', page - 1, 0, requestId);
+ });
+ }
+ return Promise.resolve(requestId);
},
/** @override */
@@ -136,17 +149,8 @@ cr.define('print_preview', function() {
},
/** Stubs for |print_preview.NativeLayer| methods that call C++ handlers. */
- previewReadyForTest: function() {},
startHideDialog: function () {},
- /** @return {!cr.EventTarget} The native layer event target. */
- getEventTarget: function() { return this.eventTarget_; },
-
- /** @param {!cr.EventTarget} eventTarget The event target to use. */
- setEventTarget: function(eventTarget) {
- this.eventTarget_ = eventTarget;
- },
-
/**
* @param {!print_preview.NativeInitialSettings} settings The settings
* to return as a response to |getInitialSettings|.
« no previous file with comments | « chrome/browser/ui/webui/print_preview/print_preview_ui.cc ('k') | chrome/test/data/webui/print_preview/plugin_stub.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698