Title: [203153] trunk
- Revision
- 203153
- Author
- [email protected]
- Date
- 2016-07-12 23:44:58 -0700 (Tue, 12 Jul 2016)
Log Message
[Fetch API] isRedirected should be conveyed in workers
https://bugs.webkit.org/show_bug.cgi?id=159676
Patch by Youenn Fablet <[email protected]> on 2016-07-12
Reviewed by Alex Christensen.
LayoutTests/imported/w3c:
* web-platform-tests/fetch/api/redirect/redirect-method-worker-expected.txt:
Source/WebCore:
Passing isRedirected value between threads.
Rebasing corresponding worker test, even though it is currently skipped (due to crashing flakiness).
* platform/network/ResourceResponseBase.cpp:
(WebCore::ResourceResponseBase::crossThreadData):
(WebCore::ResourceResponseBase::fromCrossThreadData):
* platform/network/ResourceResponseBase.h:
Modified Paths
Diff
Modified: trunk/LayoutTests/imported/w3c/ChangeLog (203152 => 203153)
--- trunk/LayoutTests/imported/w3c/ChangeLog 2016-07-13 05:45:02 UTC (rev 203152)
+++ trunk/LayoutTests/imported/w3c/ChangeLog 2016-07-13 06:44:58 UTC (rev 203153)
@@ -1,3 +1,12 @@
+2016-07-12 Youenn Fablet <[email protected]>
+
+ [Fetch API] isRedirected should be conveyed in workers
+ https://bugs.webkit.org/show_bug.cgi?id=159676
+
+ Reviewed by Alex Christensen.
+
+ * web-platform-tests/fetch/api/redirect/redirect-method-worker-expected.txt:
+
2016-07-08 Daniel Bates <[email protected]>
Setting table.tFoot or calling table.createTFoot() should append HTML tfont element to the end of the table
Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/redirect/redirect-method-worker-expected.txt (203152 => 203153)
--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/redirect/redirect-method-worker-expected.txt 2016-07-13 05:45:02 UTC (rev 203152)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/redirect/redirect-method-worker-expected.txt 2016-07-13 06:44:58 UTC (rev 203153)
@@ -1,4 +1,5 @@
+PASS Response.redirected should be false on not-redirected responses
PASS Redirect 301 with GET
PASS Redirect 301 with POST
PASS Redirect 301 with HEAD
Modified: trunk/Source/WebCore/ChangeLog (203152 => 203153)
--- trunk/Source/WebCore/ChangeLog 2016-07-13 05:45:02 UTC (rev 203152)
+++ trunk/Source/WebCore/ChangeLog 2016-07-13 06:44:58 UTC (rev 203153)
@@ -1,3 +1,18 @@
+2016-07-12 Youenn Fablet <[email protected]>
+
+ [Fetch API] isRedirected should be conveyed in workers
+ https://bugs.webkit.org/show_bug.cgi?id=159676
+
+ Reviewed by Alex Christensen.
+
+ Passing isRedirected value between threads.
+ Rebasing corresponding worker test, even though it is currently skipped (due to crashing flakiness).
+
+ * platform/network/ResourceResponseBase.cpp:
+ (WebCore::ResourceResponseBase::crossThreadData):
+ (WebCore::ResourceResponseBase::fromCrossThreadData):
+ * platform/network/ResourceResponseBase.h:
+
2016-07-12 Eric Carlson <[email protected]>
REGRESSION (r202509): media controls controls enabled AirPlay placeholder is shown
Modified: trunk/Source/WebCore/platform/network/ResourceResponseBase.cpp (203152 => 203153)
--- trunk/Source/WebCore/platform/network/ResourceResponseBase.cpp 2016-07-13 05:45:02 UTC (rev 203152)
+++ trunk/Source/WebCore/platform/network/ResourceResponseBase.cpp 2016-07-13 06:44:58 UTC (rev 203153)
@@ -73,6 +73,7 @@
data.httpHeaderFields = httpHeaderFields().isolatedCopy();
data.resourceLoadTiming = m_resourceLoadTiming.isolatedCopy();
data.type = m_type;
+ data.isRedirected = m_isRedirected;
return data;
}
@@ -93,6 +94,7 @@
response.m_httpHeaderFields = WTFMove(data.httpHeaderFields);
response.m_resourceLoadTiming = data.resourceLoadTiming;
response.m_type = data.type;
+ response.m_isRedirected = data.isRedirected;
return response;
}
Modified: trunk/Source/WebCore/platform/network/ResourceResponseBase.h (203152 => 203153)
--- trunk/Source/WebCore/platform/network/ResourceResponseBase.h 2016-07-13 05:45:02 UTC (rev 203152)
+++ trunk/Source/WebCore/platform/network/ResourceResponseBase.h 2016-07-13 06:44:58 UTC (rev 203153)
@@ -59,6 +59,7 @@
HTTPHeaderMap httpHeaderFields;
ResourceLoadTiming resourceLoadTiming;
Type type;
+ bool isRedirected;
};
CrossThreadData crossThreadData() const;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
