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

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

Issue 736883002: Implement <select> Popup Menu using PagePopup (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years 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 9f2ebd16e6e1fdc367b383a97a0c07fcc9b004e7..9557ef4af211f84927f2e91d6e73efa8f5ef6716 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
@@ -21,6 +21,11 @@ function test()
eventSender.keyDown('downArrow', ['altKey']);
shouldBeTrue('internals.isSelectPopupVisible(popup)');
eventSender.keyDown('downArrow', ['altKey']);
+ // FIXME: 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');
@@ -28,6 +33,11 @@ function test()
eventSender.keyDown('upArrow', ['altKey']);
shouldBeTrue('internals.isSelectPopupVisible(popup)');
eventSender.keyDown('upArrow', ['altKey']);
+ // FIXME: 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();

Powered by Google App Engine
This is Rietveld 408576698