Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: f08767a2f3c07d942088e8e2fa20321d2fb85f0f
https://github.com/WebKit/WebKit/commit/f08767a2f3c07d942088e8e2fa20321d2fb85f0f
Author: Chris Dumez <[email protected]>
Date: 2026-05-11 (Mon, 11 May 2026)
Changed paths:
M LayoutTests/platform/ios/TestExpectations
M LayoutTests/platform/mac-wk2/TestExpectations
M Source/WebKit/NetworkProcess/Classifier/WebResourceLoadStatisticsStore.cpp
Log Message:
-----------
REGRESSION(304892@main?):
http/tests/resourceLoadStatistics/log-cross-site-load-with-link-decoration.html
is a flakey text failure
https://bugs.webkit.org/show_bug.cgi?id=305508
rdar://168172161
Reviewed by Brent Fulgham.
304892@main defers old web process shutdown during commitProvisionalPage via a
shutdownPreventingScope. This allows the old process to receive the Close
message
and flush its (potentially empty) resource load statistics to the network
process.
resourceLoadStatisticsUpdated() unconditionally calls
processStatisticsAndDataRecords()
even when there is nothing to merge. This processing iterates all domains and
resets
DataRemovalFrequency to Never for any domain that was recently flagged for
removal,
racing with logCrossSiteLoadWithLinkDecoration() which just set it to Short.
The fix is to early-return when the statistics vector is empty since there is
nothing
to merge and no reason to trigger data records processing.
No new tests, unskip existing test.
* LayoutTests/platform/ios/TestExpectations:
* LayoutTests/platform/mac-wk2/TestExpectations:
* Source/WebKit/NetworkProcess/Classifier/WebResourceLoadStatisticsStore.cpp:
(WebKit::WebResourceLoadStatisticsStore::resourceLoadStatisticsUpdated):
Canonical link: https://commits.webkit.org/313039@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
