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

Unified Diff: third_party/WebKit/LayoutTests/fast/events/touch/touch-input-element-change-documents.html

Issue 1895303007: Non passive touch end or touch cancel listeners should not block scroll. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2704
Patch Set: Created 4 years, 8 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: third_party/WebKit/LayoutTests/fast/events/touch/touch-input-element-change-documents.html
diff --git a/third_party/WebKit/LayoutTests/fast/events/touch/touch-input-element-change-documents.html b/third_party/WebKit/LayoutTests/fast/events/touch/touch-input-element-change-documents.html
index 8ccd6cf2e690bb649d554b063b45aa52f8bbc7e2..1eddc0ddc91ba3eb87b3b91f5eb7b85becfcf6e6 100644
--- a/third_party/WebKit/LayoutTests/fast/events/touch/touch-input-element-change-documents.html
+++ b/third_party/WebKit/LayoutTests/fast/events/touch/touch-input-element-change-documents.html
@@ -4,26 +4,29 @@
description("This test checks that we correctly update the touch event handler count when an Input element with default touch handlers changes documents.");
- shouldBe('window.internals.touchEventHandlerCount(document)', '0');
+ shouldBe('window.internals.touchStartOrMoveEventHandlerCount(document)', '0');
// There are two touchable Input elements in Audio's shadow DOM when controls are enabled.
var input = document.createElement('audio');
input.setAttribute("controls", true);
var container = document.getElementById('container');
- shouldBe('window.internals.touchEventHandlerCount(document)', '0');
+ shouldBe('window.internals.touchStartOrMoveEventHandlerCount(document)', '0');
container.appendChild(input);
- shouldBe('window.internals.touchEventHandlerCount(document)', '2');
+ shouldBe('window.internals.touchStartOrMoveEventHandlerCount(document)', '2');
+ shouldBe('window.internals.touchEndOrCancelEventHandlerCount(document)', '0');
input.ontouchend = function() { };
- shouldBe('window.internals.touchEventHandlerCount(document)', '3');
+ shouldBe('window.internals.touchStartOrMoveEventHandlerCount(document)', '2');
+ shouldBe('window.internals.touchEndOrCancelEventHandlerCount(document)', '1');
input.ontouchend = null;
- shouldBe('window.internals.touchEventHandlerCount(document)', '2');
+ shouldBe('window.internals.touchStartOrMoveEventHandlerCount(document)', '2');
+ shouldBe('window.internals.touchEndOrCancelEventHandlerCount(document)', '0');
document.implementation.createDocument("", "", null).adoptNode(input);
- shouldBe('window.internals.touchEventHandlerCount(document)', '0');
+ shouldBe('window.internals.touchStartOrMoveEventHandlerCount(document)', '0');
</script>

Powered by Google App Engine
This is Rietveld 408576698