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

Unified Diff: third_party/WebKit/LayoutTests/fast/events/pointerevents/mouse-pointer-updown-events.html

Issue 1391393002: Fire pointerup/down events for corresponding mouse events. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 2 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: third_party/WebKit/LayoutTests/fast/events/pointerevents/mouse-pointer-updown-events.html
diff --git a/third_party/WebKit/LayoutTests/fast/events/pointerevents/mouse-pointer-updown-events.html b/third_party/WebKit/LayoutTests/fast/events/pointerevents/mouse-pointer-updown-events.html
new file mode 100644
index 0000000000000000000000000000000000000000..06f0b6b34619b46fb9918a9dde35eacd82376011
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/fast/events/pointerevents/mouse-pointer-updown-events.html
@@ -0,0 +1,88 @@
+<!DOCTYPE HTML>
+<script src="../../../resources/js-test.js"></script>
+<style>
+div {
+ margin: 10px;
+ padding: 50px;
+ float: left;
+}
+#console {
+ padding: 0px;
+ float: none;
Rick Byers 2015/10/20 19:09:50 curious why you need these rules. If the console
mustaq 2015/10/21 15:17:21 I try to make the console divs sane to ease quick
Rick Byers 2015/10/22 21:31:51 Yeah that makes sense, just wondering who the heck
mustaq 2015/10/23 14:59:44 The generic 'div' style I set above is the culprit
+}
+</style>
+
+<div id="green" style="background-color:green">
+</div>
+
+<div id="blue" style="background-color:blue">
+</div>
+
+<div id="console"></div>
+
+<script>
+description("Verifies that pointerup/down are fired correctly for correponding mouse events.");
+
+var eventToPreventDefault = "";
Rick Byers 2015/10/20 19:09:50 Isn't this really redundant with your preventDefau
mustaq 2015/10/21 15:17:21 Done, left only the tests for bubbled events here.
+
+function init() {
+ var eventList = ["mousedown", "mouseup", "pointerdown", "pointerup"];
+
+ ["green", "blue"].forEach(function(id) {
+ var targetDiv = document.getElementById(id);
+
+ eventList.forEach(function(eventName) {
+ targetDiv.addEventListener(eventName, function(event) {
+ if (event.eventPhase != Event.AT_TARGET)
+ return;
+
+ var preventDefaultMsg = "";
+ if (event.type == eventToPreventDefault && id == "blue") {
+ event.preventDefault();
+ preventDefaultMsg = ", prevented default";
+ }
+
+ debug(id + " received " + event.type + preventDefaultMsg);
+ });
+ });
+ });
+}
+
+function runTests() {
+ var rect = document.getElementById("green").getBoundingClientRect();
+ var x1 = rect.left + 5;
+ var y1 = rect.top + 5;
+
+ var rect = document.getElementById("blue").getBoundingClientRect();
+ var x2 = rect.left + 5;
+ var y2 = rect.top + 5;
+
+ var preventDefaultEventList = ["none", "pointerup", "pointerdown"];
+
+ preventDefaultEventList.forEach(function(preventDefaultEvent) {
+ debug("--- test with preventDefault on " + preventDefaultEvent + "---");
+ eventToPreventDefault = preventDefaultEvent;
+
+ eventSender.mouseMoveTo(x1, y1);
+ eventSender.mouseDown();
+
+ eventSender.mouseMoveTo(x2, y2);
+ eventSender.mouseUp();
+ eventSender.mouseDown();
+
+ eventSender.mouseMoveTo(x1, y1);
+ eventSender.mouseUp();
+ eventSender.mouseDown();
+ eventSender.mouseUp();
+
+ debug("");
+ });
+}
+
+init();
+if (window.eventSender)
+ runTests();
+else
+ debug("This test requires eventSender");
+
+</script>

Powered by Google App Engine
This is Rietveld 408576698