[webkit-changes] [WebKit/WebKit] cc94cc: fast/scrolling/ios/click-events-during-momentum-sc...

Wenson Hsieh noreply at github.com
Tue May 2 11:22:41 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: cc94cce0848d72816737efdb85a3d469dd7e2cc3
      https://github.com/WebKit/WebKit/commit/cc94cce0848d72816737efdb85a3d469dd7e2cc3
  Author: Wenson Hsieh <wenson_hsieh at apple.com>
  Date:   2023-05-02 (Tue, 02 May 2023)

  Changed paths:
    M LayoutTests/fast/scrolling/ios/click-events-during-momentum-scroll-in-main-frame-expected.txt
    M LayoutTests/fast/scrolling/ios/click-events-during-momentum-scroll-in-main-frame.html
    M LayoutTests/fast/scrolling/ios/click-events-during-momentum-scroll-in-overflow-expected.txt
    M LayoutTests/fast/scrolling/ios/click-events-during-momentum-scroll-in-overflow.html

  Log Message:
  -----------
  fast/scrolling/ios/click-events-during-momentum-scroll-in-main-frame.html is a flaky failure
https://bugs.webkit.org/show_bug.cgi?id=256192

Reviewed by Aditya Keerthi.

Adjust these two tests to (hopefully) be less flaky. These tests try to scroll via pan gesture, and
then tap in a web view during the deceleration phase of the scroll animation; however, it's
apparently possible for the test to synthesize the tap gesture while scrolling is still happening
(due to the pan gesture), reusing the touch identifier corresponding to the touch used for
scrolling.

Make this test more realistic (and also easier to follow) by removing the `noteTestProgress()`
counter mechanism, and instead simply wait for the scroll position to pass a certain threshold after
the event stream has ended.

Before this adjustment, I was able to reliably reproduce 1-2 test failures every 200 runs; after
this adjustment, I'm no longer able to reproduce any flaky failures.

* LayoutTests/fast/scrolling/ios/click-events-during-momentum-scroll-in-main-frame-expected.txt:
* LayoutTests/fast/scrolling/ios/click-events-during-momentum-scroll-in-main-frame.html:
* LayoutTests/fast/scrolling/ios/click-events-during-momentum-scroll-in-overflow-expected.txt:
* LayoutTests/fast/scrolling/ios/click-events-during-momentum-scroll-in-overflow.html:

Canonical link: https://commits.webkit.org/263599@main




More information about the webkit-changes mailing list