Attachment 346494 Details for Bug 188163 – Patch

[patch] Patch

bug-188163-20180803180421.patch (text/plain), 2.19 KB, created by Rob Buis on 2018-08-03 09:04:22 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Rob Buis

Created: 2018-08-03 09:04:22 PDT

Size: 2.19 KB

patch

obsolete

>Subversion Revision: 234544
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 11ff0f572c837655993468f02d4e46f0d4f5d651..fd0185e5a481285fcf7ee047a4a8fcb11349b182 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,16 @@
>+2018-08-03  Rob Buis  <rbuis@igalia.com>
>+
>+        [GTK][WPE] Fetch tests assert in SubresourceLoader::didReceiveResponse()
>+        https://bugs.webkit.org/show_bug.cgi?id=188163
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Fetch in manual redirect mode uses didReceiveResponse instead of
>+        willSendRequestInternal, so update the ASSERT.
>+
>+        * loader/SubresourceLoader.cpp:
>+        (WebCore::SubresourceLoader::didReceiveResponse):
>+
> 2018-08-03  Zalan Bujtas  <zalan@apple.com>
> 
>         [LFC] Do not check margin box while validating geometry.
>diff --git a/Source/WebCore/loader/SubresourceLoader.cpp b/Source/WebCore/loader/SubresourceLoader.cpp
>index 2cf70fa44f6625cfc17eb20b84391af60fd522e5..9a4dd30ee8e664ab251a11df06cfd864b0a1c118 100644
>--- a/Source/WebCore/loader/SubresourceLoader.cpp
>+++ b/Source/WebCore/loader/SubresourceLoader.cpp
>@@ -318,11 +318,11 @@ void SubresourceLoader::didReceiveResponse(const ResourceResponse& response, Com
>     }
> #endif
> 
>-    // We want redirect responses to be processed through willSendRequestInternal.
>-    // The only exception is redirection with no Location headers. Or in rare circumstances,
>+    // We want redirect responses to be processed through willSendRequestInternal. Exceptions are
>+    // redirection with no Location headers and fetch in manual redirect mode. Or in rare circumstances,
>     // cases of too many redirects from CFNetwork (<rdar://problem/30610988>).
> #if !PLATFORM(COCOA)
>-    ASSERT(response.httpStatusCode() < 300 || response.httpStatusCode() >= 400 || response.httpStatusCode() == 304 || !response.httpHeaderField(HTTPHeaderName::Location));
>+    ASSERT(response.httpStatusCode() < 300 || response.httpStatusCode() >= 400 || response.httpStatusCode() == 304 || !response.httpHeaderField(HTTPHeaderName::Location) || response.type() == ResourceResponse::Type::Opaqueredirect);
> #endif
> 
>     // Reference the object in this method since the additional processing can do

Actions: View | Formatted Diff | Diff

Attachments on bug 188163: 346494 | 346514