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/resources/picker-common.js

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/resources/picker-common.js
diff --git a/LayoutTests/fast/forms/resources/picker-common.js b/LayoutTests/fast/forms/resources/picker-common.js
index 0a20ef231c5d5a32e95adcb13a10f11f955cb73b..f3733a05f9dad6fde9afd7326537b759f2795959 100644
--- a/LayoutTests/fast/forms/resources/picker-common.js
+++ b/LayoutTests/fast/forms/resources/picker-common.js
@@ -19,14 +19,18 @@ function sendKey(input, keyName, ctrlKey, altKey) {
input.dispatchEvent(event);
}
-function openPicker(input, callback) {
+function openPicker(element, callback) {
window.moveTo();
- input.offsetTop; // Force to lay out
- if (input.type === "color") {
- input.focus();
- eventSender.keyDown(" ");
- } else {
- sendKey(input, "Down", false, true);
+ element.offsetTop; // Force to lay out
+ if (element.tagName === "SELECT") {
+ sendKey(element, "Down", false, true);
+ } else if (element.tagName === "INPUT") {
+ if (element.type === "color") {
+ element.focus();
+ eventSender.keyDown(" ");
+ } else {
+ sendKey(element, "Down", false, true);
+ }
}
popupWindow = window.internals.pagePopupWindow;
if (typeof callback === "function") {

Powered by Google App Engine
This is Rietveld 408576698