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

Unified Diff: third_party/WebKit/LayoutTests/fast/events/mouseenter-mouseleave-on-drag.html

Issue 1402133003: Skip updating nodeUnderMouse to clickTarget. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added a test. 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/fast/events/mouseenter-mouseleave-on-drag-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/fast/events/mouseenter-mouseleave-on-drag.html
diff --git a/third_party/WebKit/LayoutTests/fast/events/mouseenter-mouseleave-on-drag.html b/third_party/WebKit/LayoutTests/fast/events/mouseenter-mouseleave-on-drag.html
index 1a5207ff835cfd6c7f3e35ea623aaaf461af1714..05c4342a561a77b81648c173594a16c2e15d723b 100644
--- a/third_party/WebKit/LayoutTests/fast/events/mouseenter-mouseleave-on-drag.html
+++ b/third_party/WebKit/LayoutTests/fast/events/mouseenter-mouseleave-on-drag.html
@@ -26,7 +26,7 @@
<div id="console"></div>
<script>
-description("Verifies that dragging mouse in/out of an element doesn't fire redundant mouseenter or mouseleave events (crbug.com/470258)");
+description("Verifies that dragging mouse in/out of an element doesn't fire redundant mouseenter or mouseleave events (crbug.com/356090 & crbug.com/470258)");
var wasInside = {};
@@ -48,31 +48,47 @@ function init() {
wasInside[id] = false;
});
- targetDiv.addEventListener("mousedown", function(event) {
- debug(id + " received mousedown");
- });
-
- targetDiv.addEventListener("mouseup", function(event) {
- debug(id + " received mouseup");
+ ["mouseover", "mouseout", "mousedown", "mouseup"].forEach(function(eventName) {
+ targetDiv.addEventListener(eventName, function(event) {
+ if (event.eventPhase == Event.AT_TARGET)
+ debug(id + " received " + eventName);
+ });
});
});
}
init();
if (window.eventSender) {
- eventSender.mouseMoveTo(1, 150);
+ var y = 150;
- eventSender.mouseMoveTo(110, 150);
+ debug("--- drag across inner ---");
+ eventSender.mouseMoveTo(1, y);
+ eventSender.mouseMoveTo(110, y);
eventSender.mouseDown();
+ eventSender.mouseMoveTo(140, y);
+ eventSender.mouseMoveTo(150, y);
+ eventSender.mouseMoveTo(160, y);
+ eventSender.mouseMoveTo(190, y);
+ eventSender.mouseUp();
+ eventSender.mouseMoveTo(201, y);
- eventSender.mouseMoveTo(140, 150);
- eventSender.mouseMoveTo(150, 150);
- eventSender.mouseMoveTo(160, 150);
+ debug("");
- eventSender.mouseMoveTo(190, 150);
+ debug("--- drag into inner ---");
+ eventSender.mouseMoveTo(190, y);
+ eventSender.mouseDown();
+ eventSender.mouseMoveTo(150, y);
eventSender.mouseUp();
- eventSender.mouseMoveTo(201, 150);
+ debug("");
+
+ debug("--- drag out of inner ---");
+ eventSender.mouseDown();
+ eventSender.mouseMoveTo(110, y);
+ eventSender.mouseUp();
+
+ debug("");
+
} else {
debug("This test requires eventSender");
}
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/fast/events/mouseenter-mouseleave-on-drag-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698