[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