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

Unified Diff: LayoutTests/fast/forms/select/menulist-popup-open-hide-using-keyboard.html

Issue 892083003: Revert Implement <select> Popup Menu using PagePopup (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 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: LayoutTests/fast/forms/select/menulist-popup-open-hide-using-keyboard.html
diff --git a/LayoutTests/fast/forms/select/menulist-popup-open-hide-using-keyboard.html b/LayoutTests/fast/forms/select/menulist-popup-open-hide-using-keyboard.html
index c5656703427b1304ea443e63f70217cc9789af7d..9f2ebd16e6e1fdc367b383a97a0c07fcc9b004e7 100644
--- a/LayoutTests/fast/forms/select/menulist-popup-open-hide-using-keyboard.html
+++ b/LayoutTests/fast/forms/select/menulist-popup-open-hide-using-keyboard.html
@@ -8,8 +8,6 @@
<option value="two">Two</option>
</select>
<script>
-window.jsTestIsAsync = true;
-
function test()
{
description('Test for opening select popup and closing.');
@@ -23,11 +21,6 @@ function test()
eventSender.keyDown('downArrow', ['altKey']);
shouldBeTrue('internals.isSelectPopupVisible(popup)');
eventSender.keyDown('downArrow', ['altKey']);
- // We need this delay because in ListPicker._handleKeyDown we hide the
- // popup with a delay to avoid reopening the popup.
- setTimeout(test2, 0);
-}
-function test2() {
shouldBeFalse('internals.isSelectPopupVisible(popup)');
debug('Test opening and closing of popup using alt and up arrow key');
@@ -35,11 +28,6 @@ function test2() {
eventSender.keyDown('upArrow', ['altKey']);
shouldBeTrue('internals.isSelectPopupVisible(popup)');
eventSender.keyDown('upArrow', ['altKey']);
- // We need this delay because in ListPicker._handleKeyDown we hide the
- // popup with a delay to avoid reopening the popup.
- setTimeout(test3, 0);
-}
-function test3() {
shouldBeFalse('internals.isSelectPopupVisible(popup)');
popup.blur();
@@ -71,7 +59,6 @@ function test3() {
shouldBeTrue('internals.isSelectPopupVisible(popup)');
popup.blur();
- finishJSTest();
}
test();
</script>
« no previous file with comments | « LayoutTests/fast/forms/resources/picker-common.js ('k') | LayoutTests/fast/forms/select/popup-menu-appearance.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698