Chromium Code Reviews| Index: chrome/test/data/webui/print_preview_destination_search_test.js |
| diff --git a/chrome/test/data/webui/print_preview_destination_search_test.js b/chrome/test/data/webui/print_preview_destination_search_test.js |
| index 2155bd7820beb4ff844b9bbad2fb318f5ba96494..c284da68f2d91525c0525544a70a3daffb019d56 100644 |
| --- a/chrome/test/data/webui/print_preview_destination_search_test.js |
| +++ b/chrome/test/data/webui/print_preview_destination_search_test.js |
| @@ -196,7 +196,6 @@ TEST_F('PrintPreviewDestinationSearchTest', 'Select', function() { |
| }); |
| test('ReceiveSuccessfulSetup', function() { |
| - |
| var destId = "00112233DEADBEEF"; |
| var waiter = waitForEvent( |
| @@ -217,6 +216,11 @@ TEST_F('PrintPreviewDestinationSearchTest', 'Select', function() { |
| }); |
| test('ReceiveFailedSetup', function() { |
|
dpapad
2017/05/02 00:07:41
if (cr.isChromeOS) {
test('ReceiveFailedSetup',
skau
2017/05/02 01:52:48
Done.
|
| + if (!cr.isChromeOS) { |
| + // The 'ResolutionFails' test covers this case for non-CrOS. |
|
dpapad
2017/05/02 00:07:41
ResolutionFails test also returns early for non-Cr
skau
2017/05/02 01:52:48
The comment is accurate. The other test is disabl
|
| + return; |
| + } |
| + |
| var destId = '00112233DEADBEEF'; |
| var resolver = mockSetupCall(destId, nativeLayer_); |
| @@ -225,14 +229,8 @@ TEST_F('PrintPreviewDestinationSearchTest', 'Select', function() { |
| // Force resolution to fail. |
| resolveSetup(resolver, destId, false, null); |
| - if (cr.isChromeOS) { |
| - // Selection should not change on ChromeOS. |
| - assertEquals(null, destinationStore_.selectedDestination); |
| - } else { |
| - // Other code expects selection to be present so it still occurs |
| - // for non-CrOS. |
| - assertEquals(destId, destinationStore_.selectedDestination.id); |
| - } |
| + // Selection should not change on ChromeOS. |
| + assertEquals(null, destinationStore_.selectedDestination); |
| }); |
| }); |