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

Unified Diff: LayoutTests/fast/events/resources/mouse-events-within-no-element-iframe.html

Issue 494913003: Mousemove event should be fired when the mouse stays in the document but there is no element. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 4 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: LayoutTests/fast/events/resources/mouse-events-within-no-element-iframe.html
diff --git a/LayoutTests/fast/events/resources/mouse-events-within-no-element-iframe.html b/LayoutTests/fast/events/resources/mouse-events-within-no-element-iframe.html
new file mode 100644
index 0000000000000000000000000000000000000000..6de526de3bb86a0dc47bfc1be322c5d6cd8b6981
--- /dev/null
+++ b/LayoutTests/fast/events/resources/mouse-events-within-no-element-iframe.html
@@ -0,0 +1,72 @@
+<html>
+<head>
+<style type="text/css">
+ html, body, .workspace{
+ height: 100%;
+ }
+ button {
+ cursor: pointer;
+ padding: 0px;
+ margin: 0px;
+ }
+ .minbox {
+ height: 30px;
+ width: 1px;
+ }
+#div1, #div2 {
+height: 80px;
+border: 1px solid #aaaaaa;
+}
+</style>
+</head>
+<body>
+<div class="workspace"></div>
+<div class="minbox">
+ <div id="test">
+ <button>POINTER</button>
+ </div>
+</div>
+<script>
Julien - ping for review 2014/09/08 20:07:41 This test is super difficult to analyze for severa
Miyoung Shin(g) 2014/09/11 23:53:28 I'm changing the test case but there is a flaky is
+ function eventHandler(e) {
+ alert(e.type + " : " + e.currentTarget);
+ };
+ window.scrollBy(0, 200);
+ if (window.testRunner) {
+ var div = document.getElementById("test");
+ var x = div.offsetLeft;
+ var y = div.offsetTop-window.pageYOffset+10;
+ x=x+200;
Julien - ping for review 2014/09/08 20:07:41 You did a good job at following Blink's style belo
Miyoung Shin(g) 2014/09/11 23:53:28 I will fix it.
+
+ document.addEventListener("mousemove", eventHandler);
+ document.body.addEventListener("mousemove", eventHandler);
+ eventSender.mouseMoveTo(x, y);
+ eventSender.mouseMoveTo(x, y + 100);
+ eventSender.mouseMoveTo(x, y - 100);
+ eventSender.mouseMoveTo(x, y + 100);
+ document.removeEventListener("mousemove", eventHandler, false);
+ document.body.removeEventListener("mousemove", eventHandler, false);
+
+ document.addEventListener("mouseover", eventHandler);
+ eventSender.mouseMoveTo(x, y);
+ eventSender.mouseMoveTo(x, y + 100);
+ document.removeEventListener("mouseover", eventHandler, false);
+
+ document.addEventListener("mouseout", eventHandler);
+ eventSender.mouseMoveTo(x, y);
+ eventSender.mouseMoveTo(x, y + 100);
+ document.removeEventListener("mouseout", eventHandler, false);
+
+ document.addEventListener("mouseenter", eventHandler);
+ document.addEventListener("mouseleave", eventHandler);
+ document.body.addEventListener("mouseenter", eventHandler);
+ document.body.addEventListener("mouseleave", eventHandler);
+ eventSender.mouseMoveTo(x, y);
+ eventSender.mouseMoveTo(x, y + 100);
+ eventSender.mouseMoveTo(x, y - 100);
+ eventSender.mouseMoveTo(x, y + 100);
+
+ testRunner.dumpAsText();
+ }
+</script>
+</body>
+</html>
« no previous file with comments | « LayoutTests/fast/events/mouse-events-within-no-element-expected.txt ('k') | Source/core/rendering/RenderLayer.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698