tree b2ba060119092202e5b4e8abf5f89c47e8fe51f2
parent c391f6a901d2d5bffd82a80a6b981b644c683075
author David Awogbemila <awogbemila@chromium.org> 1681474381 -0700
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1681475160 -0700

Handle nop scroll overscroll-behavior edge case

crrev.com/c/4362538 ensured that nop scrolls on the layout viewport do not
trigger scrollend events on the document. It did not explicitly handle
nop scrolls on all scrollers because it assumed that an inner
scroller that can't consume the delta would bubble the scroll up to the
document but in the case of non-auto overscroll-behvior-{x,y} even if
the scroller can't consume the delta the scroll will not bubble up to the
document/its parent.

This patch also updates
scrollend-event-fired-to-element-with-overscroll-behavior.html not to
expect scrollend when there is no scroll but continues to ensure that we
do not bubble a scroll up to the document for non-auto overscroll
behavior.
This test expectedly failed on wpt.fyi and timed out on the trybots (the
timeout was locally reproducible). With this patch, I no longer see the
timeout so I am removing the TestExpectation entry.
The test file could probably use a better name now that it expects that
scrollend is *not* fired but I am not chaning it so it's easier to see
its pass/fail history.

Bug: 1430947
Change-Id: Ie528cd5a001ece6ed54d6163c536e5774b07c39f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4418186
Reviewed-by: Steve Kobes <skobes@chromium.org>
Code-Coverage: Findit <findit-for-me@appspot.gserviceaccount.com>
Commit-Queue: David Awogbemila <awogbemila@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1130387}