[webkit-changes] [WebKit/WebKit] 35d631: REGRESSION(285688 at main?): [ macOS iOS wk2 ] 15x im...
Alex Christensen
noreply at github.com
Thu Dec 19 22:06:58 PST 2024
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 35d63141717ff9968cb65f69665f884dd33f6cc4
https://github.com/WebKit/WebKit/commit/35d63141717ff9968cb65f69665f884dd33f6cc4
Author: Alex Christensen <achristensen at apple.com>
Date: 2024-12-19 (Thu, 19 Dec 2024)
Changed paths:
M LayoutTests/platform/ios/TestExpectations
M LayoutTests/platform/mac-wk2/TestExpectations
M Tools/WebKitTestRunner/TestController.cpp
Log Message:
-----------
REGRESSION(285688 at main?): [ macOS iOS wk2 ] 15x imported/w3c/web-platform-tests/upgrade-insecure-requests/gen/* are near constant failures.
https://bugs.webkit.org/show_bug.cgi?id=282128
rdar://138680837
Reviewed by Charlie Wolfe.
285688 at main removed a call to Frame::disownOpener and replaced it by another
call to Frame::disownOpener that happens between the tests, but the one we removed
happened after about:blank is loaded and the one we added happens before.
This is fine most of the time, but Frame::disownOpener also calls
LocalFrame::reinitializeDocumentSecurityContext which needs to happen after we
load about:blank between the tests. This was evident in security context state
being left behind between tests. If you ran this test:
imported/w3c/web-platform-tests/upgrade-insecure-requests/gen/sharedworker-module-data.http-rp/upgrade/fetch.https.html
immediately followed by this test:
imported/w3c/web-platform-tests/upgrade-insecure-requests/gen/srcdoc-inherit.meta/unset/fetch.https.html
the state left behind would cause the second test to fail. Moving the call to
WKPageResetStateBetweenTests a few lines later after we have loaded about:blank
fixes this and makes the second test no longer fail from state left behind.
* LayoutTests/platform/ios/TestExpectations:
* LayoutTests/platform/mac-wk2/TestExpectations:
* Tools/WebKitTestRunner/TestController.cpp:
(WTR::TestController::resetStateToConsistentValues):
Canonical link: https://commits.webkit.org/288147@main
To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications
More information about the webkit-changes
mailing list