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

Unified Diff: trunk/LayoutTests/fast/forms/listbox-onchange.html

Issue 1184693003: Revert 196987 "Don't invoke default actions for MouseEvents gene..." (Closed) Base URL: svn://svn.chromium.org/blink/
Patch Set: Created 5 years, 6 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: trunk/LayoutTests/fast/forms/listbox-onchange.html
===================================================================
--- trunk/LayoutTests/fast/forms/listbox-onchange.html (revision 197019)
+++ trunk/LayoutTests/fast/forms/listbox-onchange.html (working copy)
@@ -122,46 +122,34 @@
function mouseDownOnSelect(selId, index, shift, metaOrCtrl)
{
- modifiers = [];
- if (shift)
- modifiers[0] = "shiftKey";
+ var meta = false;
+ var ctrl = false;
if (metaOrCtrl) {
if (navigator.userAgent.search(/\bMac OS X\b/) != -1)
- modifiers[modifiers.length] = "metaKey";
+ meta = true;
else
- modifiers[modifiers.length] = "ctrlKey";
+ ctrl = true;
}
-
var sl = document.getElementById(selId);
- var rect = sl.options[index].getBoundingClientRect();
- if (window.eventSender) {
- eventSender.mouseMoveTo(rect.left + 1, rect.top + 1);
- eventSender.mouseDown(0, modifiers);
- } else {
- debug("FAIL: This test requires window.eventSender.");
- }
+ var event = document.createEvent("MouseEvent");
+ event.initMouseEvent("mousedown", true, true, document.defaultView, 1, 0, 0, 0, 0, ctrl, false, shift, meta, 0, document);
+ sl.options[index].dispatchEvent(event);
}
function mouseUpOnSelect(selId, index, shift, metaOrCtrl)
{
- modifiers = [];
- if (shift)
- modifiers[0] = "shiftKey";
+ var meta = false;
+ var ctrl = false;
if (metaOrCtrl) {
if (navigator.userAgent.search(/\bMac OS X\b/) != -1)
- modifiers[modifiers.length] = "metaKey";
+ meta = true;
else
- modifiers[modifiers.length] = "ctrlKey";
+ ctrl = true;
}
-
var sl = document.getElementById(selId);
- var rect = sl.options[index].getBoundingClientRect();
- if (window.eventSender) {
- eventSender.mouseMoveTo(rect.left + 1, rect.top + 1);
- eventSender.mouseUp(0, modifiers);
- } else {
- debug("FAIL: This test requires window.eventSender.");
- }
+ var event = document.createEvent("MouseEvent");
+ event.initMouseEvent("mouseup", true, true, document.defaultView, 1, 0, 0, 0, 0, ctrl, false, shift, meta, 0, document);
+ sl.options[index].dispatchEvent(event);
}
function keyDownOnSelect(selId, identifier, shift, metaOrCtrl)
@@ -173,7 +161,7 @@
if (navigator.userAgent.search(/\bMac OS X\b/) != -1)
modifiers[modifiers.length] = "metaKey";
else
- modifiers[modifiers.length] = "ctrlKey";
+ modifiers[modifiers.length] = "controlKey";
}
document.getElementById(selId).focus();

Powered by Google App Engine
This is Rietveld 408576698