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

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

Issue 2537573003: Merge "SELECT element: Commit a provisional OPTION selection on blur." to M56 (Closed)
Patch Set: Created 4 years, 1 month 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: third_party/WebKit/LayoutTests/fast/forms/select/select-disabled.html
diff --git a/third_party/WebKit/LayoutTests/fast/forms/select/select-disabled.html b/third_party/WebKit/LayoutTests/fast/forms/select/select-disabled.html
index d6165cfe26629b5401d6952a91c73aec87845e51..ea24225a3062b8f76cc6ace41e57e186f3c1b3b3 100644
--- a/third_party/WebKit/LayoutTests/fast/forms/select/select-disabled.html
+++ b/third_party/WebKit/LayoutTests/fast/forms/select/select-disabled.html
@@ -32,6 +32,7 @@
<script>
description('Tests that user cannot change disabled select by keyboard');
+jsTestIsAsync = true;
var select1 = document.getElementById('select1');
select1.focus();
@@ -60,18 +61,20 @@ var select3 = document.getElementById('select3');
clickElement(select3);
shouldBeTrue("internals.isSelectPopupVisible(select3)");
select3.disabled = true;
-shouldBeFalse("internals.isSelectPopupVisible(select3)");
-
-debug('Tests on select disable, value cannot be changed');
-var select4 = document.getElementById('select4');
-select4.focus();
-eventSender.keyDown('b');
-shouldBeEqualToString('select4.value', 'b');
-select4.disabled = true;
-eventSender.keyDown('c');
-shouldBeEqualToString('select4.value', 'b');
-
+select3.offsetLeft;
+setTimeout(() => {
+ shouldBeFalse("internals.isSelectPopupVisible(select3)");
+ debug('Tests on select disable, value cannot be changed');
+ var select4 = document.getElementById('select4');
+ select4.focus();
+ eventSender.keyDown('b');
+ shouldBeEqualToString('select4.value', 'b');
+ select4.disabled = true;
+ eventSender.keyDown('c');
+ shouldBeEqualToString('select4.value', 'b');
+ finishJSTest();
+}, 0);
</script>
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698