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

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

Issue 2606043004: Perform printer setup on Chrome OS before selecting printer. (Closed)
Patch Set: fix unit test 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
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 5848d82ad20217a23a73956fc439c2146cf8736b..fe4afab3565f846a5ae3f9b809e80635b089480b 100644
--- a/chrome/test/data/webui/print_preview.js
+++ b/chrome/test/data/webui/print_preview.js
@@ -387,14 +387,17 @@ TEST_F('PrintPreviewWebUITest', 'TestPrintPreviewRestoreLocalDestination',
//Test with multiple destinations
TEST_F('PrintPreviewWebUITest', 'TestPrintPreviewRestoreMultipleDestinations',
function() {
+ var origin = cr.isChromeOS ? "chrome_os" : "local";
+
this.initialSettings_.serializedAppStateStr_ =
dpapad 2017/01/12 01:18:10 Nit (optional): Instead of painfully constructing
skau 2017/01/12 02:17:17 Done.
- '{"version":2,"recentDestinations":[{"id":"ID1", "origin":"local",' +
+ '{"version":2,"recentDestinations":[{' +
+ '"id":"ID1", "origin":"' + origin + '",' +
'"account":"", "capabilities":0, "name":"", "extensionId":"",' +
'"extensionName":""},' +
- '{"id":"ID2", "origin":"local",' +
+ '{"id":"ID2", "origin":"' + origin + '",' +
'"account":"", "capabilities":0, "name":"", "extensionId":"",' +
'"extensionName":""},' +
- '{"id":"ID3", "origin":"local",' +
+ '{"id":"ID3", "origin":"' + origin + '",' +
'"account":"", "capabilities":0, "name":"", "extensionId":"",' +
'"extensionName":""}]}';
this.setInitialSettings();

Powered by Google App Engine
This is Rietveld 408576698