Chromium Code Reviews Chromium Code Reviews
Issue 2812613004:
Fire a click event even when a clicked text node is removed in mouseup (Closed)
| Index: third_party/WebKit/LayoutTests/fast/events/remove-text-node-in-mouseup.html |
| diff --git a/third_party/WebKit/LayoutTests/fast/events/remove-text-node-in-mouseup.html b/third_party/WebKit/LayoutTests/fast/events/remove-text-node-in-mouseup.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..66adaddc8f126c5668f27bc97386a53b5baf84eb |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/fast/events/remove-text-node-in-mouseup.html |
| @@ -0,0 +1,27 @@ |
| +<!DOCTYPE html> |
| +<p>This test ensures Blink does fire a click event on the parent element of a clicked text node |
| +even when the text node has been removed in mouseup event.</p> |
| +<div id="test"></div> |
|
yosin_UTC9
2017/04/12 06:10:44
It is better to use w3c test handler to avoid havi
hayato
2017/04/12 06:16:06
We can't because the test needs eventSender.
kochi
2017/04/12 06:44:39
I think it's okay to use testharness.js for tests
hayato
2017/04/12 06:52:22
It would be, but I would prefer consistency with o
|
| +<div id="target" style="display:inline-block" onmouseup="mouseup()" onclick="test.innerHTML = 'PASS';">Click Here</div> |
| +<script> |
| +const test = document.getElementById('test'); |
| +const target = document.getElementById('target'); |
| + |
| +function mouseup() { |
| + test.innerHTML = 'FAIL'; |
| + target.firstChild.remove(); |
| +} |
| + |
| +if (window.testRunner) { |
| + testRunner.dumpAsText(); |
| + if (!window.eventSender) { |
| + test.innerHTML = 'FAIL - this test requires eventSender'; |
| + } else { |
| + eventSender.mouseMoveTo(target.offsetLeft + target.offsetWidth / 2, |
| + target.offsetTop + target.offsetHeight / 2); |
| + eventSender.mouseDown(); |
| + eventSender.leapForward(200); |
| + eventSender.mouseUp(); |
| + } |
| +} |
| +</script> |
