Chromium Code Reviews| Index: chrome/test/data/webui/settings/dropdown_menu_tests.js |
| diff --git a/chrome/test/data/webui/settings/dropdown_menu_tests.js b/chrome/test/data/webui/settings/dropdown_menu_tests.js |
| index 77c3768ded422a0a4a0752bf58fe4f5f3ec5307a..a10a09f51d98e939201e9f0b450c79b98dcd095c 100644 |
| --- a/chrome/test/data/webui/settings/dropdown_menu_tests.js |
| +++ b/chrome/test/data/webui/settings/dropdown_menu_tests.js |
| @@ -92,6 +92,33 @@ cr.define('settings_dropdown_menu', function() { |
| // Pref should not have changed. |
| assertEquals('f', dropdown.pref.value); |
| }); |
| + |
| + test('delay setting options', function testDelayedOptions(done) { |
| + dropdown.pref = { |
| + key: 'test.number2', |
| + type: chrome.settingsPrivate.PrefType.NUMBER, |
| + value: 200, |
| + }; |
| + |
|
dschuyler
2016/08/17 23:59:46
Would it make sense to have check that
the label i
|
| + setTimeout(function() { |
| + assertEquals( |
| + loadTimeData.getValue('loading'), dropdown.$.dropdownMenu.label); |
|
dschuyler
2016/08/17 23:59:46
...that the label is not shown here?
I think we w
|
| + assertTrue(dropdown.$.dropdownMenu.disabled); |
| + assertEquals(undefined, selectable.selected); |
| + |
| + dropdown.menuOptions = [{value: 100, name: 'Option 100'}, |
| + {value: 200, name: 'Option 200'}, |
| + {value: 300, name: 'Option 300'}, |
| + {value: 400, name: 'Option 400'}]; |
| + Polymer.dom.flush(); |
| + |
| + // Dropdown menu enables itself and selects the new menu option |
| + // correpsonding to the pref value. |
| + assertFalse(dropdown.$.dropdownMenu.disabled); |
| + assertEquals('200', selectable.selected); |
| + done(); |
| + }, 100); |
| + }); |
| }); |
| } |