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

Unified Diff: LayoutTests/fast/events/touch/gesture/gesture-tap-div-removed.html

Issue 429793004: Re-add extra hit-tests on GestureTap (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix patch Created 6 years, 5 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/touch/gesture/gesture-tap-div-removed.html
diff --git a/LayoutTests/fast/events/touch/gesture/gesture-tap-div-removed.html b/LayoutTests/fast/events/touch/gesture/gesture-tap-div-removed.html
index 2a1d646cd5dde3511d43a139b84a428342b3735f..576072fe604ad60bc51bfb11f35d663d5ea04302 100644
--- a/LayoutTests/fast/events/touch/gesture/gesture-tap-div-removed.html
+++ b/LayoutTests/fast/events/touch/gesture/gesture-tap-div-removed.html
@@ -12,10 +12,6 @@
Tap me
</div>
<script>
-
-if (window.testRunner)
- testRunner.dumpAsText();
-
function eventLogger(e) {
debug("Received " + e.type + " on " + (e.currentTarget.id || e.currentTarget.nodeName));
}
@@ -38,13 +34,13 @@ function getTargetPoint() {
// so that it's elligble for garbage collection.
var target = document.getElementById('target');
target.addEventListener('mousemove', logAndRemove);
- target.addEventListener('mousedown', eventLogger);
- target.addEventListener('mouseup', eventLogger);
- target.addEventListener('click', eventLogger);
+ target.addEventListener('mousedown', logAndFail);
+ target.addEventListener('mouseup', logAndFail);
+ target.addEventListener('click', logAndFail);
document.addEventListener('mousemove', eventLogger);
- document.addEventListener('mousedown', logAndFail);
- document.addEventListener('mouseup', logAndFail);
- document.addEventListener('click', logAndFail);
+ document.addEventListener('mousedown', eventLogger);
+ document.addEventListener('mouseup', eventLogger);
+ document.addEventListener('click', eventLogger);
var rect = target.getBoundingClientRect();
return {
@@ -54,7 +50,7 @@ function getTargetPoint() {
}
onload = function() {
- description("Verifies that deleting the element being tapped during the event sequence doesn't cause any problems. Succeeds if the expected events are delivered to the div, events after removal don't bubble up, and we don't crash.");
+ description("Verifies that deleting the element being tapped during the event sequence doesn't cause any problems. Succeeds if the expected events are delivered, events after removal go to the new node at the point of tap, and we don't crash.");
var point = getTargetPoint();

Powered by Google App Engine
This is Rietveld 408576698