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

Side by Side Diff: chrome/test/data/webui/print_preview/print_preview_tests.js

Issue 2948723002: Print Preview: Change print to cr.sendWithPromise (Closed)
Patch Set: Address comments 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 cr.define('print_preview_test', function() { 5 cr.define('print_preview_test', function() {
6 /** 6 /**
7 * Index of the "Save as PDF" printer. 7 * Index of the "Save as PDF" printer.
8 * @type {number} 8 * @type {number}
9 * @const 9 * @const
10 */ 10 */
(...skipping 1289 matching lines...) Expand 10 before | Expand all | Expand 10 after
1300 return nativeLayer.whenCalled('getPrinterCapabilities', 'BarDevice') 1300 return nativeLayer.whenCalled('getPrinterCapabilities', 'BarDevice')
1301 .then(function() { 1301 .then(function() {
1302 // Dispatch event indicating new preview has loaded. 1302 // Dispatch event indicating new preview has loaded.
1303 var previewDoneEvent = new Event( 1303 var previewDoneEvent = new Event(
1304 print_preview.PreviewArea.EventType.PREVIEW_GENERATION_DONE); 1304 print_preview.PreviewArea.EventType.PREVIEW_GENERATION_DONE);
1305 previewArea.dispatchEvent(previewDoneEvent); 1305 previewArea.dispatchEvent(previewDoneEvent);
1306 1306
1307 // Has active print button and successfully 'prints', indicating 1307 // Has active print button and successfully 'prints', indicating
1308 // recovery from error state. 1308 // recovery from error state.
1309 expectFalse(printButton.disabled); 1309 expectFalse(printButton.disabled);
1310 expectFalse(nativeLayer.isPrintStarted());
1311 printButton.click(); 1310 printButton.click();
1312 expectTrue(nativeLayer.isPrintStarted()); 1311 // This should result in a call to print.
1312 return nativeLayer.whenCalled('print');
dpapad 2017/06/23 00:13:57 Related to comment in native_layer_stub.js, here w
1313 }); 1313 });
1314 }); 1314 });
1315 }); 1315 });
1316 1316
1317 // Test the preview generator to make sure the generate draft parameter is 1317 // Test the preview generator to make sure the generate draft parameter is
1318 // set correctly. It should be false if the only change is the page range. 1318 // set correctly. It should be false if the only change is the page range.
1319 test('GenerateDraft', function() { 1319 test('GenerateDraft', function() {
1320 // Use a real preview generator. 1320 // Use a real preview generator.
1321 previewArea.previewGenerator_ = 1321 previewArea.previewGenerator_ =
1322 new print_preview.PreviewGenerator(printPreview.destinationStore_, 1322 new print_preview.PreviewGenerator(printPreview.destinationStore_,
1323 printPreview.printTicketStore_, nativeLayer, 1323 printPreview.printTicketStore_, nativeLayer,
1324 printPreview.documentInfo_); 1324 printPreview.documentInfo_);
1325 return setupSettingsAndDestinationsWithCapabilities().then(function() { 1325 return setupSettingsAndDestinationsWithCapabilities().then(function() {
1326 // The first request should generate draft because there was no 1326 // The first request should generate draft because there was no
1327 // previous print preview draft. 1327 // previous print preview draft.
1328 expectTrue(nativeLayer.generateDraft()); 1328 expectTrue(nativeLayer.generateDraft());
1329 1329
1330 // Change the page range - no new draft needed. 1330 // Change the page range - no new draft needed.
1331 printPreview.printTicketStore_.pageRange.updateValue('2'); 1331 printPreview.printTicketStore_.pageRange.updateValue('2');
1332 expectFalse(nativeLayer.generateDraft()); 1332 expectFalse(nativeLayer.generateDraft());
1333 1333
1334 // Change the margin type - need to regenerate again. 1334 // Change the margin type - need to regenerate again.
1335 printPreview.printTicketStore_.marginsType.updateValue( 1335 printPreview.printTicketStore_.marginsType.updateValue(
1336 print_preview.ticket_items.MarginsTypeValue.NO_MARGINS); 1336 print_preview.ticket_items.MarginsTypeValue.NO_MARGINS);
1337 expectTrue(nativeLayer.generateDraft()); 1337 expectTrue(nativeLayer.generateDraft());
1338 }); 1338 });
1339 }); 1339 });
1340 }); 1340 });
1341 }); 1341 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698