Unreviewed, rolling out r215814.
authorryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 26 Apr 2017 23:52:19 +0000 (23:52 +0000)
committerryanhaddad@apple.com <ryanhaddad@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 26 Apr 2017 23:52:19 +0000 (23:52 +0000)
The LayoutTest for this change is failing on ios-simulator and
is flaky on macOS.

Reverted changeset:

"Response.blob() does not set the content-type based on the
header value."
https://bugs.webkit.org/show_bug.cgi?id=170849
http://trac.webkit.org/changeset/215814

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@215842 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/imported/w3c/ChangeLog
LayoutTests/imported/w3c/web-platform-tests/fetch/api/response/response-consume-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/Modules/fetch/FetchResponse.cpp

index 37db074..4d6a47a 100644 (file)
@@ -1,3 +1,17 @@
+2017-04-26  Ryan Haddad  <ryanhaddad@apple.com>
+
+        Unreviewed, rolling out r215814.
+
+        The LayoutTest for this change is failing on ios-simulator and
+        is flaky on macOS.
+
+        Reverted changeset:
+
+        "Response.blob() does not set the content-type based on the
+        header value."
+        https://bugs.webkit.org/show_bug.cgi?id=170849
+        http://trac.webkit.org/changeset/215814
+
 2017-04-26  Chris Dumez  <cdumez@apple.com>
 
         Re-sync html/browsers/the-window-object web-platform-tests from upstream
index ec97801..e8fdb31 100644 (file)
@@ -33,7 +33,7 @@ FAIL Consume response's body: from stream with correct multipart type to formDat
 FAIL Consume response's body: from stream without correct multipart type to formData (error case) assert_throws: function "function () { throw e }" threw "Not implemented" (undefined) expected object "TypeError" ("TypeError")
 FAIL Consume response's body: from stream with correct urlencoded type to formData promise_test: Unhandled rejection with value: "Not implemented"
 FAIL Consume response's body: from stream without correct urlencoded type to formData (error case) assert_throws: function "function () { throw e }" threw "Not implemented" (undefined) expected object "TypeError" ("TypeError")
-PASS Consume response's body: from fetch to blob 
+FAIL Consume response's body: from fetch to blob assert_equals: Blob body type should be computed from the response Content-Type expected "text/plain" but got ""
 PASS Consume response's body: from fetch to text 
 PASS Consume response's body: from fetch to arrayBuffer 
 FAIL Consume response's body: from fetch without correct type to formData (error case) assert_throws: function "function () { throw e }" threw "Not implemented" (undefined) expected object "TypeError" ("TypeError")
index 8c7ff50..22441e8 100644 (file)
@@ -1,3 +1,17 @@
+2017-04-26  Ryan Haddad  <ryanhaddad@apple.com>
+
+        Unreviewed, rolling out r215814.
+
+        The LayoutTest for this change is failing on ios-simulator and
+        is flaky on macOS.
+
+        Reverted changeset:
+
+        "Response.blob() does not set the content-type based on the
+        header value."
+        https://bugs.webkit.org/show_bug.cgi?id=170849
+        http://trac.webkit.org/changeset/215814
+
 2017-04-26  Wenson Hsieh  <wenson_hsieh@apple.com>
 
         WebItemProviderPasteboard should not synchronously load provided data
index 4098e48..ac7d7e4 100644 (file)
@@ -173,7 +173,6 @@ void FetchResponse::BodyLoader::didReceiveResponse(const ResourceResponse& resou
 
     m_response.m_response = resourceResponse;
     m_response.m_headers->filterAndFill(resourceResponse.httpHeaderFields(), FetchHeaders::Guard::Response);
-    m_response.updateContentType();
 
     std::exchange(m_promise, std::nullopt)->resolve(m_response);
 }