Diff
Modified: releases/WebKitGTK/webkit-2.20/LayoutTests/ChangeLog (229244 => 229245)
--- releases/WebKitGTK/webkit-2.20/LayoutTests/ChangeLog 2018-03-05 11:21:52 UTC (rev 229244)
+++ releases/WebKitGTK/webkit-2.20/LayoutTests/ChangeLog 2018-03-05 11:22:04 UTC (rev 229245)
@@ -1,3 +1,18 @@
+2018-02-26 Commit Queue <[email protected]>
+
+ Unreviewed, rolling out r226745.
+ https://bugs.webkit.org/show_bug.cgi?id=183132
+
+ This is breaking some websites (Requested by youenn on
+ #webkit).
+
+ Reverted changeset:
+
+ "Use no-cache fetch mode when loading main documents with
+ location.reload()"
+ https://bugs.webkit.org/show_bug.cgi?id=181285
+ https://trac.webkit.org/changeset/226745
+
2018-02-25 Fujii Hironori <[email protected]>
[WK2] http/tests/navigation/new-window-redirect-history.html crashes
Modified: releases/WebKitGTK/webkit-2.20/LayoutTests/http/tests/inspector/network/har/har-page-expected.txt (229244 => 229245)
--- releases/WebKitGTK/webkit-2.20/LayoutTests/http/tests/inspector/network/har/har-page-expected.txt 2018-03-05 11:21:52 UTC (rev 229244)
+++ releases/WebKitGTK/webkit-2.20/LayoutTests/http/tests/inspector/network/har/har-page-expected.txt 2018-03-05 11:22:04 UTC (rev 229245)
@@ -43,10 +43,10 @@
"cookies": [],
"headers": "<filtered>",
"content": {
- "size": "<filtered>",
+ "size": 3042,
"compression": 0,
"mimeType": "text/html",
- "text": "<filtered text (3171)>"
+ "text": "<filtered text (3042)>"
},
"redirectURL": "",
"headersSize": "<filtered>",
@@ -84,7 +84,7 @@
"cookies": [],
"headers": "<filtered>",
"content": {
- "size": "<filtered>",
+ "size": 0,
"compression": 0,
"mimeType": "application/x-_javascript_",
"text": "<filtered text (7039)>"
Modified: releases/WebKitGTK/webkit-2.20/LayoutTests/http/tests/inspector/network/har/har-page.html (229244 => 229245)
--- releases/WebKitGTK/webkit-2.20/LayoutTests/http/tests/inspector/network/har/har-page.html 2018-03-05 11:21:52 UTC (rev 229244)
+++ releases/WebKitGTK/webkit-2.20/LayoutTests/http/tests/inspector/network/har/har-page.html 2018-03-05 11:22:04 UTC (rev 229245)
@@ -42,9 +42,6 @@
// Reduce the file contents.
if (key === "text")
return "<filtered text (" + value.length + ")>";
- // Reduce the file size since cache may or may not be used.
- if (key === "size")
- return "<filtered>";
// Since cache may or may not be used, timing data may be variable.
// NOTE: SSL should always be -1 for this test case.
Modified: releases/WebKitGTK/webkit-2.20/LayoutTests/imported/w3c/ChangeLog (229244 => 229245)
--- releases/WebKitGTK/webkit-2.20/LayoutTests/imported/w3c/ChangeLog 2018-03-05 11:21:52 UTC (rev 229244)
+++ releases/WebKitGTK/webkit-2.20/LayoutTests/imported/w3c/ChangeLog 2018-03-05 11:22:04 UTC (rev 229245)
@@ -1,3 +1,18 @@
+2018-02-26 Commit Queue <[email protected]>
+
+ Unreviewed, rolling out r226745.
+ https://bugs.webkit.org/show_bug.cgi?id=183132
+
+ This is breaking some websites (Requested by youenn on
+ #webkit).
+
+ Reverted changeset:
+
+ "Use no-cache fetch mode when loading main documents with
+ location.reload()"
+ https://bugs.webkit.org/show_bug.cgi?id=181285
+ https://trac.webkit.org/changeset/226745
+
2018-02-16 Daniel Bates <[email protected]>
Remove UTF-32 BOM parsing code
Modified: releases/WebKitGTK/webkit-2.20/LayoutTests/imported/w3c/web-platform-tests/service-workers/service-worker/fetch-event.https-expected.txt (229244 => 229245)
--- releases/WebKitGTK/webkit-2.20/LayoutTests/imported/w3c/web-platform-tests/service-workers/service-worker/fetch-event.https-expected.txt 2018-03-05 11:21:52 UTC (rev 229244)
+++ releases/WebKitGTK/webkit-2.20/LayoutTests/imported/w3c/web-platform-tests/service-workers/service-worker/fetch-event.https-expected.txt 2018-03-05 11:22:04 UTC (rev 229245)
@@ -1,4 +1,5 @@
+
PASS Service Worker headers in the request of a fetch event
PASS Service Worker responds to fetch event with string
PASS Service Worker responds to fetch event with blob body
@@ -11,7 +12,7 @@
PASS Multiple calls of respondWith must throw InvalidStateErrors
PASS Service Worker event.respondWith must set the used flag
PASS Service Worker should expose FetchEvent URL fragments.
-PASS Service Worker responds to fetch event with the correct cache types
+FAIL Service Worker responds to fetch event with the correct cache types assert_unreached: unexpected rejection: assert_equals: expected "no-cache" but got "default" Reached unreachable code
PASS Service Worker should intercept EventSource
PASS Service Worker responds to fetch event with the correct integrity_metadata
PASS FetchEvent#body is a string
Modified: releases/WebKitGTK/webkit-2.20/Source/WebCore/ChangeLog (229244 => 229245)
--- releases/WebKitGTK/webkit-2.20/Source/WebCore/ChangeLog 2018-03-05 11:21:52 UTC (rev 229244)
+++ releases/WebKitGTK/webkit-2.20/Source/WebCore/ChangeLog 2018-03-05 11:22:04 UTC (rev 229245)
@@ -1,3 +1,18 @@
+2018-02-26 Commit Queue <[email protected]>
+
+ Unreviewed, rolling out r226745.
+ https://bugs.webkit.org/show_bug.cgi?id=183132
+
+ This is breaking some websites (Requested by youenn on
+ #webkit).
+
+ Reverted changeset:
+
+ "Use no-cache fetch mode when loading main documents with
+ location.reload()"
+ https://bugs.webkit.org/show_bug.cgi?id=181285
+ https://trac.webkit.org/changeset/226745
+
2018-02-24 Fujii Hironori <[email protected]>
Null-dereference of the second argument `resource` of DocumentLoader::scheduleSubstituteResourceLoad
Modified: releases/WebKitGTK/webkit-2.20/Source/WebCore/loader/DocumentLoader.cpp (229244 => 229245)
--- releases/WebKitGTK/webkit-2.20/Source/WebCore/loader/DocumentLoader.cpp 2018-03-05 11:21:52 UTC (rev 229244)
+++ releases/WebKitGTK/webkit-2.20/Source/WebCore/loader/DocumentLoader.cpp 2018-03-05 11:22:04 UTC (rev 229245)
@@ -1669,32 +1669,9 @@
});
}
-static inline FetchOptions::Cache toFetchOptionsCache(ResourceRequestCachePolicy policy)
-{
- // We are setting FetchOptions::Cache values to keep current behavior consistency.
- // FIXME: We should merge FetchOptions::Cache with ResourceRequestCachePolicy and merge related class members.
- switch (policy) {
- case UseProtocolCachePolicy:
- return FetchOptions::Cache::Default;
- case ReloadIgnoringCacheData:
- return FetchOptions::Cache::Reload;
- case ReturnCacheDataElseLoad:
- return FetchOptions::Cache::Default;
- case ReturnCacheDataDontLoad:
- return FetchOptions::Cache::Default;
- case DoNotUseAnyCache:
- return FetchOptions::Cache::NoStore;
- case RefreshAnyCacheData:
- return FetchOptions::Cache::NoCache;
- }
- return FetchOptions::Cache::Default;
-}
-
void DocumentLoader::loadMainResource(ResourceRequest&& request)
{
- ResourceLoaderOptions mainResourceLoadOptions { SendCallbacks, SniffContent, BufferData, StoredCredentialsPolicy::Use, ClientCredentialPolicy::MayAskClientForCredentials, FetchOptions::Credentials::Include, SkipSecurityCheck, FetchOptions::Mode::Navigate, IncludeCertificateInfo, ContentSecurityPolicyImposition::SkipPolicyCheck, DefersLoadingPolicy::AllowDefersLoading, CachingPolicy::AllowCaching };
- mainResourceLoadOptions.cache = toFetchOptionsCache(request.cachePolicy());
-
+ static NeverDestroyed<ResourceLoaderOptions> mainResourceLoadOptions(SendCallbacks, SniffContent, BufferData, StoredCredentialsPolicy::Use, ClientCredentialPolicy::MayAskClientForCredentials, FetchOptions::Credentials::Include, SkipSecurityCheck, FetchOptions::Mode::Navigate, IncludeCertificateInfo, ContentSecurityPolicyImposition::SkipPolicyCheck, DefersLoadingPolicy::AllowDefersLoading, CachingPolicy::AllowCaching);
CachedResourceRequest mainResourceRequest(ResourceRequest(request), mainResourceLoadOptions);
if (!m_frame->isMainFrame() && m_frame->document()) {
// If we are loading the main resource of a subframe, use the cache partition of the main document.
Modified: releases/WebKitGTK/webkit-2.20/Source/WebCore/loader/FrameLoader.cpp (229244 => 229245)
--- releases/WebKitGTK/webkit-2.20/Source/WebCore/loader/FrameLoader.cpp 2018-03-05 11:21:52 UTC (rev 229244)
+++ releases/WebKitGTK/webkit-2.20/Source/WebCore/loader/FrameLoader.cpp 2018-03-05 11:22:04 UTC (rev 229245)
@@ -1191,12 +1191,10 @@
referrer = String();
FrameLoadType loadType;
- if (request.resourceRequest().cachePolicy() == RefreshAnyCacheData)
+ if (request.resourceRequest().cachePolicy() == ReloadIgnoringCacheData)
loadType = FrameLoadType::Reload;
else if (request.lockBackForwardList() == LockBackForwardList::Yes)
loadType = FrameLoadType::RedirectWithLockedBackForwardList;
- else if (request.resourceRequest().cachePolicy() == ReloadIgnoringCacheData)
- loadType = FrameLoadType::ReloadFromOrigin;
else
loadType = FrameLoadType::Standard;
@@ -1286,7 +1284,7 @@
addExtraFieldsToRequest(request, newLoadType, true);
if (isReload(newLoadType))
- request.setCachePolicy(RefreshAnyCacheData);
+ request.setCachePolicy(ReloadIgnoringCacheData);
ASSERT(newLoadType != FrameLoadType::Same);
@@ -1423,7 +1421,7 @@
FrameLoadType type;
if (shouldTreatURLAsSameAsCurrent(newDocumentLoader->originalRequest().url())) {
- r.setCachePolicy(RefreshAnyCacheData);
+ r.setCachePolicy(ReloadIgnoringCacheData);
type = FrameLoadType::Same;
} else if (shouldTreatURLAsSameAsCurrent(newDocumentLoader->unreachableURL()) && m_loadType == FrameLoadType::Reload)
type = FrameLoadType::Reload;
@@ -1648,7 +1646,8 @@
ResourceRequest& request = loader->request();
- request.setCachePolicy(RefreshAnyCacheData);
+ // FIXME: We don't have a mechanism to revalidate the main resource without reloading at the moment.
+ request.setCachePolicy(ReloadIgnoringCacheData);
// If we're about to re-post, set up action so the application can warn the user.
if (request.httpMethod() == "POST")
@@ -2623,13 +2622,13 @@
if (isMainResource) {
if (isReload(loadType) || request.isConditional())
- return loadType == FrameLoadType::ReloadFromOrigin ? ReloadIgnoringCacheData : RefreshAnyCacheData;
+ return ReloadIgnoringCacheData;
return UseProtocolCachePolicy;
}
if (request.isConditional())
- return RefreshAnyCacheData;
+ return ReloadIgnoringCacheData;
if (documentLoader()->isLoadingInAPISense()) {
// If we inherit cache policy from a main resource, we use the DocumentLoader's
@@ -3489,10 +3488,8 @@
} else {
switch (loadType) {
case FrameLoadType::Reload:
+ case FrameLoadType::ReloadFromOrigin:
case FrameLoadType::ReloadExpiredOnly:
- request.setCachePolicy(RefreshAnyCacheData);
- break;
- case FrameLoadType::ReloadFromOrigin:
request.setCachePolicy(ReloadIgnoringCacheData);
break;
case FrameLoadType::Back:
Modified: releases/WebKitGTK/webkit-2.20/Source/WebCore/loader/NavigationScheduler.cpp (229244 => 229245)
--- releases/WebKitGTK/webkit-2.20/Source/WebCore/loader/NavigationScheduler.cpp 2018-03-05 11:21:52 UTC (rev 229244)
+++ releases/WebKitGTK/webkit-2.20/Source/WebCore/loader/NavigationScheduler.cpp 2018-03-05 11:22:04 UTC (rev 229245)
@@ -183,7 +183,7 @@
UserGestureIndicator gestureIndicator { userGestureToForward() };
bool refresh = equalIgnoringFragmentIdentifier(frame.document()->url(), url());
- ResourceRequest resourceRequest { url(), referrer(), refresh ? RefreshAnyCacheData : UseProtocolCachePolicy };
+ ResourceRequest resourceRequest { url(), referrer(), refresh ? ReloadIgnoringCacheData : UseProtocolCachePolicy };
FrameLoadRequest frameLoadRequest { initiatingDocument(), *securityOrigin(), resourceRequest, "_self", lockHistory(), lockBackForwardList(), MaybeSendReferrer, AllowNavigationToInvalidURL::No, NewFrameOpenerPolicy::Allow, shouldOpenExternalURLs(), initiatedByMainFrame() };
frame.loader().changeLocation(WTFMove(frameLoadRequest));
@@ -217,7 +217,7 @@
{
UserGestureIndicator gestureIndicator { userGestureToForward() };
- ResourceRequest resourceRequest { url(), referrer(), RefreshAnyCacheData };
+ ResourceRequest resourceRequest { url(), referrer(), ReloadIgnoringCacheData };
FrameLoadRequest frameLoadRequest { initiatingDocument(), *securityOrigin(), resourceRequest, "_self", lockHistory(), lockBackForwardList(), MaybeSendReferrer, AllowNavigationToInvalidURL::Yes, NewFrameOpenerPolicy::Allow, shouldOpenExternalURLs(), initiatedByMainFrame() };
frame.loader().changeLocation(WTFMove(frameLoadRequest));
@@ -320,7 +320,7 @@
ResourceResponse replacementResponse { m_originDocument.url(), ASCIILiteral("text/plain"), 0, ASCIILiteral("UTF-8") };
SubstituteData replacementData { SharedBuffer::create(), m_originDocument.url(), replacementResponse, SubstituteData::SessionHistoryVisibility::Hidden };
- ResourceRequest resourceRequest { m_originDocument.url(), emptyString(), RefreshAnyCacheData };
+ ResourceRequest resourceRequest { m_originDocument.url(), emptyString(), ReloadIgnoringCacheData };
FrameLoadRequest frameLoadRequest { m_originDocument, m_originDocument.securityOrigin(), resourceRequest, { }, lockHistory(), lockBackForwardList(), MaybeSendReferrer, AllowNavigationToInvalidURL::Yes, NewFrameOpenerPolicy::Allow, shouldOpenExternalURLs(), initiatedByMainFrame() };
frameLoadRequest.setSubstituteData(replacementData);
frame.loader().load(WTFMove(frameLoadRequest));
