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

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

Issue 2606043004: Perform printer setup on Chrome OS before selecting printer. (Closed)
Patch Set: fix nits Created 3 years, 11 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
« no previous file with comments | « chrome/test/BUILD.gn ('k') | chrome/test/data/webui/print_preview_destination_search_test.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/webui/print_preview.js
diff --git a/chrome/test/data/webui/print_preview.js b/chrome/test/data/webui/print_preview.js
index 1ed192e674149d91c9ae2fb8536fb8ebef68a5bd..4d7659bcd3d5f27e054e495e7ae05ad776920dd5 100644
--- a/chrome/test/data/webui/print_preview.js
+++ b/chrome/test/data/webui/print_preview.js
@@ -389,16 +389,43 @@ TEST_F('PrintPreviewWebUITest', 'TestPrintPreviewRestoreLocalDestination',
//Test with multiple destinations
TEST_F('PrintPreviewWebUITest', 'TestPrintPreviewRestoreMultipleDestinations',
function() {
- this.initialSettings_.serializedAppStateStr_ =
- '{"version":2,"recentDestinations":[{"id":"ID1", "origin":"local",' +
- '"account":"", "capabilities":0, "name":"", "extensionId":"",' +
- '"extensionName":""},' +
- '{"id":"ID2", "origin":"local",' +
- '"account":"", "capabilities":0, "name":"", "extensionId":"",' +
- '"extensionName":""},' +
- '{"id":"ID3", "origin":"local",' +
- '"account":"", "capabilities":0, "name":"", "extensionId":"",' +
- '"extensionName":""}]}';
+ var origin = cr.isChromeOS ? "chrome_os" : "local";
+
+ var appState = {
+ 'version': 2,
+ 'recentDestinations': [
+ {
+ 'id': 'ID1',
+ 'origin': origin,
+ 'account': '',
+ 'capabilities': 0,
+ 'name': '',
+ 'extensionId': '',
+ 'extensionName': ''
+ },
+ {
+ 'id': 'ID2',
+ 'origin': origin,
+ 'account': '',
+ 'capabilities': 0,
+ 'name': '',
+ 'extensionId': '',
+ 'extensionName': ''
+ },
+ {
+ 'id': 'ID3',
+ 'origin': origin,
+ 'account': '',
+ 'capabilities': 0,
+ 'name': '',
+ 'extensionId': '',
+ 'extensionName': ''
+ }
+ ]
+ };
+
+ this.initialSettings_.serializedAppStateStr_ = JSON.stringify(appState);
+
this.setInitialSettings();
// Set capabilities for the three recently used destinations + 1 more
@@ -1252,4 +1279,3 @@ TEST_F('PrintPreviewWebUITest', 'TestAdvancedSettings2Options', function() {
this.waitForAnimationToEnd('more-settings');
});
-
« no previous file with comments | « chrome/test/BUILD.gn ('k') | chrome/test/data/webui/print_preview_destination_search_test.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698