[webkit-changes] [WebKit/WebKit] 82e61f: REGRESSION(253327 at main): [ iOS ] imported/w3c/web-...

Richard Robinson noreply at github.com
Sat Nov 12 17:21:51 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 82e61f0babb2aba02d4b377262d780f556f4cbd7
      https://github.com/WebKit/WebKit/commit/82e61f0babb2aba02d4b377262d780f556f4cbd7
  Author: Richard Robinson <richard_robinson2 at apple.com>
  Date:   2022-11-12 (Sat, 12 Nov 2022)

  Changed paths:
    M LayoutTests/http/tests/scroll-to-text-fragment/no-iframe-match-expected.html
    M LayoutTests/http/tests/scroll-to-text-fragment/no-iframe-match.html
    M LayoutTests/http/tests/scroll-to-text-fragment/start-text-emoji.html
    M LayoutTests/http/tests/scroll-to-text-fragment/start-text-quote.html
    M LayoutTests/http/tests/scroll-to-text-fragment/start-text-sentence.html
    M LayoutTests/http/tests/scroll-to-text-fragment/start-text-word.html
    M LayoutTests/http/tests/scroll-to-text-fragment/word-display-none.html
    M LayoutTests/platform/ios/TestExpectations
    M Source/WebCore/page/FrameView.cpp
    M Tools/DumpRenderTree/TestOptions.cpp
    M Tools/WebKitTestRunner/TestOptions.cpp

  Log Message:
  -----------
  REGRESSION(253327 at main): [ iOS ] imported/w3c/web-platform-tests/css/css-pseudo/target-text-008.html is an almost constant failure
https://bugs.webkit.org/show_bug.cgi?id=243962
rdar://98693973

Reviewed by Aditya Keerthi and Megan Gardner.

This test is failing because the snapshot occasionally captures the text indicator.
We work around this in our own tests with the `scrollToTextFragmentIndicatorEnabled()`
setting. This is not easily an option for WPT, since WPT tests the standard
configuration.

This PR works around this by disabling the text indicator if `isControlledByAutomation`
is true, which will fix the test when it is run with the WPT infrastructure.
WebKitTestRunner will also need some of its behavior to change so that the preference
is disabled when the test is run with our own infrastructure.

* LayoutTests/http/tests/scroll-to-text-fragment/no-iframe-match-expected.html:
* LayoutTests/http/tests/scroll-to-text-fragment/no-iframe-match.html:
* LayoutTests/http/tests/scroll-to-text-fragment/start-text-emoji.html:
* LayoutTests/http/tests/scroll-to-text-fragment/start-text-quote.html:
* LayoutTests/http/tests/scroll-to-text-fragment/start-text-sentence.html:
* LayoutTests/http/tests/scroll-to-text-fragment/start-text-word.html:
* LayoutTests/http/tests/scroll-to-text-fragment/word-display-none.html:
* Source/WebCore/page/FrameView.cpp:
(WebCore::FrameView::scrollToFragment):
* Tools/DumpRenderTree/TestOptions.cpp:
(WTR::TestOptions::defaults):

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




More information about the webkit-changes mailing list