Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 3caec8396210f21ab36adc76f8c37091dc553bbc
https://github.com/WebKit/WebKit/commit/3caec8396210f21ab36adc76f8c37091dc553bbc
Author: Frédéric Wang Nélar <[email protected]>
Date: 2026-04-28 (Tue, 28 Apr 2026)
Changed paths:
M Source/WebCore/Modules/fetch/FetchResponse.cpp
M Source/WebCore/Modules/model-element/HTMLModelElement.cpp
M Source/WebCore/html/HTMLImageLoader.cpp
M Source/WebCore/inspector/NetworkResourcesData.cpp
M Source/WebCore/inspector/agents/InspectorNetworkAgent.cpp
M Source/WebCore/inspector/agents/WebConsoleAgent.cpp
M Source/WebCore/loader/CrossOriginAccessControl.cpp
M Source/WebCore/loader/DocumentLoader.cpp
M Source/WebCore/loader/HistoryController.cpp
M Source/WebCore/loader/MediaResourceLoader.cpp
M Source/WebCore/loader/NetscapePlugInStreamLoader.cpp
M Source/WebCore/loader/SubresourceLoader.cpp
M Source/WebCore/loader/icon/IconLoader.cpp
M
Source/WebCore/platform/graphics/avfoundation/cf/WebCoreAVCFResourceLoader.cpp
M
Source/WebCore/platform/graphics/avfoundation/objc/WebCoreAVFResourceLoader.mm
M Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp
M Source/WebCore/platform/network/HTTPStatusCodes.h
M Source/WebCore/platform/network/ResourceResponseBase.cpp
M Source/WebCore/platform/network/ResourceResponseBase.h
M Source/WebCore/platform/network/curl/ResourceResponse.h
M Source/WebCore/workers/WorkerFontLoadRequest.cpp
M Source/WebCore/workers/WorkerScriptLoader.cpp
M Source/WebCore/workers/service/background-fetch/BackgroundFetch.cpp
M Source/WebKit/NetworkProcess/EarlyHintsResourceLoader.cpp
M Source/WebKit/NetworkProcess/soup/NetworkDataTaskSoup.cpp
M Source/WebKit/WebProcess/InjectedBundle/API/glib/WebKitWebPage.cpp
M Source/WebKit/WebProcess/WebCoreSupport/WebResourceLoadObserver.cpp
Log Message:
-----------
Use HTTP status codes in more places
https://bugs.webkit.org/show_bug.cgi?id=313531
Reviewed by Anne van Kesteren.
This also introduces helper function based on concepts from
https://fetch.spec.whatwg.org/#statuses
Potential minor aligment with the fetch spec:
* Source/WebCore/Modules/fetch/FetchResponse.cpp: Replace isNullBodyStatus()
with isHttpNullBodyStatus(), which includes 103 Early Hints too.
(WebCore::FetchResponse::create):
(WebCore::FetchResponse::redirect):
(WebCore::isNullBodyStatus): Deleted.
* Source/WebCore/Modules/model-element/HTMLModelElement.cpp:
(WebCore::HTMLModelElement::environmentMapResourceFinished):
* Source/WebCore/html/HTMLImageLoader.cpp:
(WebCore::HTMLImageLoader::dispatchLoadEvent):
(WebCore::HTMLImageLoader::notifyFinished):
* Source/WebCore/inspector/NetworkResourcesData.cpp:
(WebCore::NetworkResourcesData::dataForURL):
* Source/WebCore/inspector/agents/InspectorNetworkAgent.cpp:
(WebCore::InspectorNetworkAgent::didReceiveResponse):
* Source/WebCore/inspector/agents/WebConsoleAgent.cpp:
(WebCore::WebConsoleAgent::didReceiveResponse):
* Source/WebCore/loader/CrossOriginAccessControl.cpp:
(WebCore::validateRangeRequestedFlag):
* Source/WebCore/loader/DocumentLoader.cpp:
(WebCore::DocumentLoader::continueAfterContentPolicy): Use
isHttpNullBodyStatus() which includes 101 Switching Protocols, 103 Early Hints
and 304 Not Modified too.
(WebCore::DocumentLoader::urlForHistoryReflectsFailure const):
* Source/WebCore/loader/HistoryController.cpp:
(WebCore::HistoryController::initializeItem):
* Source/WebCore/loader/MediaResourceLoader.cpp:
(WebCore::MediaResourceLoader::verifyMediaResponse):
* Source/WebCore/loader/NetscapePlugInStreamLoader.cpp:
(WebCore::NetscapePlugInStreamLoader::didReceiveResponse):
* Source/WebCore/loader/SubresourceLoader.cpp:
(WebCore::SubresourceLoader::didReceiveResponse):
* Source/WebCore/loader/icon/IconLoader.cpp:
(WebCore::IconLoader::notifyFinished):
*
Source/WebCore/platform/graphics/avfoundation/cf/WebCoreAVCFResourceLoader.cpp:
(WebCore::WebCoreAVCFResourceLoader::responseReceived):
*
Source/WebCore/platform/graphics/avfoundation/objc/WebCoreAVFResourceLoader.mm:
(WebCore::WebCoreAVFResourceLoader::responseReceived):
* Source/WebCore/platform/graphics/gstreamer/WebKitWebSourceGStreamer.cpp:
(CachedResourceStreamingClient::responseReceived):
* Source/WebCore/platform/network/HTTPStatusCodes.h:
(WebCore::isHttpNullBodyStatus):
(WebCore::isHttpOkStatus):
(WebCore::isHttpRangeStatus):
(WebCore::isHttpRedirectStatus):
* Source/WebCore/platform/network/ResourceResponseBase.cpp:
(WebCore::ResourceResponseBase::isSuccessful const):
(WebCore::ResourceResponseBase::isRedirection const):
* Source/WebCore/platform/network/ResourceResponseBase.h:
(WebCore::ResourceResponseBase::isRedirectionStatusCode): Deleted.
* Source/WebCore/platform/network/curl/ResourceResponse.h:
(WebCore::ResourceResponse::isMovedPermanently const):
(WebCore::ResourceResponse::isFound const):
(WebCore::ResourceResponse::isSeeOther const):
(WebCore::ResourceResponse::isUnauthorized const):
(WebCore::ResourceResponse::isProxyAuthenticationRequired const):
* Source/WebCore/workers/WorkerFontLoadRequest.cpp:
(WebCore::WorkerFontLoadRequest::didReceiveResponse):
* Source/WebCore/workers/WorkerScriptLoader.cpp:
(WebCore::WorkerScriptLoader::validateWorkerResponse):
* Source/WebCore/workers/service/background-fetch/BackgroundFetch.cpp:
(WebCore::validatePartialResponse):
(WebCore::BackgroundFetch::Record::didReceiveResponse):
* Source/WebKit/NetworkProcess/EarlyHintsResourceLoader.cpp:
(WebKit::EarlyHintsResourceLoader::handleEarlyHintsResponse):
* Source/WebKit/NetworkProcess/soup/NetworkDataTaskSoup.cpp:
(WebKit::NetworkDataTaskSoup::continueHTTPRedirection):
(WebKit::NetworkDataTaskSoup::download):
* Source/WebKit/WebProcess/InjectedBundle/API/glib/WebKitWebPage.cpp:
* Source/WebKit/WebProcess/WebCoreSupport/WebResourceLoadObserver.cpp:
(WebKit::is3xxRedirect):
Canonical link: https://commits.webkit.org/312234@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
