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

Unified Diff: LayoutTests/fast/forms/select/select-disabled.html

Issue 159743002: Select when disabled should not allow value change and popup is disabled (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Popup menu is not shown when select is disabled Created 6 years, 10 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: LayoutTests/fast/forms/select/select-disabled.html
diff --git a/LayoutTests/fast/forms/select/select-disabled.html b/LayoutTests/fast/forms/select/select-disabled.html
index 563edc9d17a61d488c170ccee7efdd9d5b8a7c44..85f8cfd76eaab737ac290beda4f3361a6af3ebcc 100644
--- a/LayoutTests/fast/forms/select/select-disabled.html
+++ b/LayoutTests/fast/forms/select/select-disabled.html
@@ -17,6 +17,14 @@
<option>c</option>
</select>
+<select id=select3>
+<option selected>a</option>
+<option>b</option>
+<option>c</option>
+</select>
+
+<input id="text"></input>
+
<script>
description('Tests that user cannot change disabled select by keyboard');
@@ -42,6 +50,20 @@ shouldBeEqualToString('select2.value', 'b');
eventSender.keyDown('c');
shouldBeEqualToString('select2.value', 'b');
+var select3 = document.getElementById('select3');
+select3.focus();
+eventSender.keyDown('downArrow');
+shouldBeEqualToString('select3.value', 'b');
+eventSender.keyDown(' ');
+shouldBeTrue("internals.isSelectPopupVisible(select3)");
+select3.blur();
tkent 2014/02/17 00:14:30 blur() closes the popup. So this test makes no se
Habib Virji 2014/02/18 10:23:21 Done. Removed blur.
+select3.disabled = true;
+select3.focus();
+eventSender.keyDown('downArrow');
+shouldBeEqualToString('select3.value', 'b');
+eventSender.keyDown(' ');
+shouldBeFalse("internals.isSelectPopupVisible(select3)");
+
</script>
</body>
</html>
« no previous file with comments | « no previous file | LayoutTests/fast/forms/select/select-disabled-expected.txt » ('j') | Source/core/html/HTMLSelectElement.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698