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

Unified Diff: LayoutTests/fast/dom/hover-after-dom-delete.html

Issue 16285002: Do not invoke or clear hover effects on node deletion when cursor is invisible (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Patch for landing Created 7 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
« no previous file with comments | « no previous file | LayoutTests/fast/dom/hover-after-dom-delete-child-invisible-cursor.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/fast/dom/hover-after-dom-delete.html
diff --git a/LayoutTests/fast/dom/hover-after-dom-delete.html b/LayoutTests/fast/dom/hover-after-dom-delete.html
new file mode 100644
index 0000000000000000000000000000000000000000..cb6cd37ff71a6332041287353064a95337260ca7
--- /dev/null
+++ b/LayoutTests/fast/dom/hover-after-dom-delete.html
@@ -0,0 +1,140 @@
+<!DOCTYPE html>
+<html>
+<head>
+<style>
+#red {
+ background-color: red;
+ position: absolute;
+ left: 50px;
+ top: 50px;
+ height: 100px;
+ width: 100px;
+}
+
+#blue {
+ background-color: blue;
+ position: absolute;
+ left: 50px;
+ top: 200px;
+ height: 100px;
+ width: 100px;
+}
+#blue:hover::after {
+ content: "Hovered.";
+}
+
+</style>
+<script src="../js/resources/js-test-pre.js"></script>
+</head>
+<body onload="runtest()" style="margin:0">
+
+<script type="text/javascript">
+var redDiv;
+var blueDiv;
+var containerDiv;
+var blueText;
+var redTextWhenHovered = "Red hovered.";
+var blueTextWhenNotHovered = "";
+var blueTextWhenHovered = "'Hovered.'";
+function runtest()
+{
+ if (!window.testRunner || !window.eventSender)
+ return;
+
+ if (!window.internals || !window.internals.setIsCursorVisible) {
+ debug("window.internals.setIsCursorVisible is required to run this test.");
+ return;
+ }
+
+ testRunner.waitUntilDone();
+
+ containerDiv = document.getElementById('container');
+
+ redDiv = document.createElement("div");
+ redDiv.id = "red";
+ blueDiv = document.createElement("div");
+ blueDiv.id = "blue";
+ insertDivs();
+
+ document.addEventListener('keydown', function(e) {
+ blueDiv.style.top = "50px";
+ redDiv.parentNode.removeChild(redDiv);
+ });
+
+ initialHoverOverRedDiv();
+
+ debug("Mouse is visible, deleting the red div.");
+ eventSender.keyDown("a");
+ window.setTimeout(testAfterDeleteCursorVisible, 0);
+}
+
+function testAfterDeleteCursorVisible()
+{
+ checkBlueHoverText(blueTextWhenHovered);
+ window.setTimeout(testWithInvisibleCursor, 0);
+}
+
+function testWithInvisibleCursor()
+{
+ blueDiv.parentNode.removeChild(blueDiv);
+ insertDivs();
+
+ initialHoverOverRedDiv();
+
+ debug("Setting the mouse cursor to be invisible.");
+ internals.setIsCursorVisible(document, false);
+ shouldBeEqualToString("redDiv.innerHTML", redTextWhenHovered);
+ shouldBeEqualToString("blueDiv.innerHTML", blueTextWhenNotHovered);
+
+ debug("Mouse is invisible, deleting the red div.");
+ eventSender.keyDown("a");
+ window.setTimeout(testAfterDeleteCursorInvisible, 0);
+}
+
+function testAfterDeleteCursorInvisible()
+{
+ checkBlueHoverText(blueTextWhenNotHovered);
+ testRunner.notifyDone();
+}
+
+
+// Helper functions
+
+function checkBlueHoverText(expectedText)
+{
+ shouldBe("blueDiv.offsetTop", "50");
+ blueText = window.getComputedStyle(document.querySelector('#blue'), ':after').content;
+ shouldBeEqualToString("blueText", expectedText);
+}
+
+function insertDivs()
+{
+ debug("Adding the red and blue divs.");
+ blueDiv.style.top = "200px";
+ containerDiv.appendChild(redDiv);
+ containerDiv.appendChild(blueDiv);
+ redDiv.addEventListener('mouseover', function(e) {
+ this.innerHTML = redTextWhenHovered;
+ });
+ shouldBe("blueDiv.offsetTop", "200");
+ shouldBe("redDiv.offsetTop", "50");
+}
+
+function initialHoverOverRedDiv()
+{
+ debug("Mouse is visible, moving it over the red div.");
+ internals.setIsCursorVisible(document, true);
+ eventSender.mouseMoveTo(100, 100);
+ shouldBeEqualToString("redDiv.innerHTML", redTextWhenHovered);
+ shouldBeEqualToString("blueDiv.innerHTML", blueTextWhenNotHovered);
+}
+</script>
+
+<div id="container">
+</div>
+
+<p>Test for <a href="http://crbug.com/240722">http://crbug.com/240722</a>. If the mouse cursor is not visible, no new hover effects should be invoked when the currently hovered node is removed from the DOM. Press any key to delete the red div.</p>
+
+<div id="console"></div>
+</body>
+</html>
« no previous file with comments | « no previous file | LayoutTests/fast/dom/hover-after-dom-delete-child-invisible-cursor.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698