<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" />
<title>[198649] trunk</title>
</head>
<body>
<style type="text/css"><!--
#msg dl.meta { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dl.meta dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer, #logmsg { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fa0 solid; padding: 6px; }
#logmsg { background: #ffc; border: 1px #fa0 solid; padding: 1em 1em 0 1em; }
#logmsg p, #logmsg pre, #logmsg blockquote { margin: 0 0 1em 0; }
#logmsg p, #logmsg li, #logmsg dt, #logmsg dd { line-height: 14pt; }
#logmsg h1, #logmsg h2, #logmsg h3, #logmsg h4, #logmsg h5, #logmsg h6 { margin: .5em 0; }
#logmsg h1:first-child, #logmsg h2:first-child, #logmsg h3:first-child, #logmsg h4:first-child, #logmsg h5:first-child, #logmsg h6:first-child { margin-top: 0; }
#logmsg ul, #logmsg ol { padding: 0; list-style-position: inside; margin: 0 0 0 1em; }
#logmsg ul { text-indent: -1em; padding-left: 1em; }#logmsg ol { text-indent: -1.5em; padding-left: 1.5em; }
#logmsg > ul, #logmsg > ol { margin: 0 0 1em 0; }
#logmsg pre { background: #eee; padding: 1em; }
#logmsg blockquote { border: 1px solid #fa0; border-left-width: 10px; padding: 1em 1em 0 1em; background: white;}
#logmsg dl { margin: 0; }
#logmsg dt { font-weight: bold; }
#logmsg dd { margin: 0; padding: 0 0 0.5em 0; }
#logmsg dd:before { content:'\00bb';}
#logmsg table { border-spacing: 0px; border-collapse: collapse; border-top: 4px solid #fa0; border-bottom: 1px solid #fa0; background: #fff; }
#logmsg table th { text-align: left; font-weight: normal; padding: 0.2em 0.5em; border-top: 1px dotted #fa0; }
#logmsg table td { text-align: right; border-top: 1px dotted #fa0; padding: 0.2em 0.5em; }
#logmsg table thead th { text-align: center; border-bottom: 1px solid #fa0; }
#logmsg table th.Corner { text-align: left; }
#logmsg hr { border: none 0; border-top: 2px dashed #fa0; height: 1px; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<div id="msg">
<dl class="meta">
<dt>Revision</dt> <dd><a href="http://trac.webkit.org/projects/webkit/changeset/198649">198649</a></dd>
<dt>Author</dt> <dd>commit-queue@webkit.org</dd>
<dt>Date</dt> <dd>2016-03-24 16:01:45 -0700 (Thu, 24 Mar 2016)</dd>
</dl>
<h3>Log Message</h3>
<pre>Unreviewed, rolling out <a href="http://trac.webkit.org/projects/webkit/changeset/198627">r198627</a>.
https://bugs.webkit.org/show_bug.cgi?id=155856
Caused use-after-free (Requested by ap on #webkit).
Reverted changeset:
"[Fetch API] Add basic loading of resources"
https://bugs.webkit.org/show_bug.cgi?id=155637
http://trac.webkit.org/changeset/198627</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkLayoutTestsChangeLog">trunk/LayoutTests/ChangeLog</a></li>
<li><a href="#trunkLayoutTestsTestExpectations">trunk/LayoutTests/TestExpectations</a></li>
<li><a href="#trunkLayoutTestsimportedw3cChangeLog">trunk/LayoutTests/imported/w3c/ChangeLog</a></li>
<li><a href="#trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicacceptheaderexpectedtxt">trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/accept-header-expected.txt</a></li>
<li><a href="#trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicintegrityexpectedtxt">trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/integrity-expected.txt</a></li>
<li><a href="#trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicmodenocorsexpectedtxt">trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/mode-no-cors-expected.txt</a></li>
<li><a href="#trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicmodesameoriginexpectedtxt">trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/mode-same-origin-expected.txt</a></li>
<li><a href="#trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicrequestforbiddenheadersexpectedtxt">trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/request-forbidden-headers-expected.txt</a></li>
<li><a href="#trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicrequestheadersexpectedtxt">trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/request-headers-expected.txt</a></li>
<li><a href="#trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicschemeaboutexpectedtxt">trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-about-expected.txt</a></li>
<li><a href="#trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicschemeblobexpectedtxt">trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob-expected.txt</a></li>
<li><a href="#trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicschemeblobworkerexpectedtxt">trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob-worker-expected.txt</a></li>
<li><a href="#trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicschemeblobjs">trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob.js</a></li>
<li><a href="#trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicschemedataexpectedtxt">trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-data-expected.txt</a></li>
<li><a href="#trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicschemeothersexpectedtxt">trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-others-expected.txt</a></li>
<li><a href="#trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicstreamresponseexpectedtxt">trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/stream-response-expected.txt</a></li>
<li><a href="#trunkSourceWebCoreChangeLog">trunk/Source/WebCore/ChangeLog</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchDOMWindowFetchcpp">trunk/Source/WebCore/Modules/fetch/DOMWindowFetch.cpp</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchDOMWindowFetchh">trunk/Source/WebCore/Modules/fetch/DOMWindowFetch.h</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchFetchBodycpp">trunk/Source/WebCore/Modules/fetch/FetchBody.cpp</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchFetchBodyh">trunk/Source/WebCore/Modules/fetch/FetchBody.h</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchFetchBodyOwnercpp">trunk/Source/WebCore/Modules/fetch/FetchBodyOwner.cpp</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchFetchBodyOwnerh">trunk/Source/WebCore/Modules/fetch/FetchBodyOwner.h</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchFetchHeaderscpp">trunk/Source/WebCore/Modules/fetch/FetchHeaders.cpp</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchFetchHeadersh">trunk/Source/WebCore/Modules/fetch/FetchHeaders.h</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchFetchLoadercpp">trunk/Source/WebCore/Modules/fetch/FetchLoader.cpp</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchFetchLoaderh">trunk/Source/WebCore/Modules/fetch/FetchLoader.h</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchFetchRequestcpp">trunk/Source/WebCore/Modules/fetch/FetchRequest.cpp</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchFetchRequesth">trunk/Source/WebCore/Modules/fetch/FetchRequest.h</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchFetchResponsecpp">trunk/Source/WebCore/Modules/fetch/FetchResponse.cpp</a></li>
<li><a href="#trunkSourceWebCoreModulesfetchFetchResponseh">trunk/Source/WebCore/Modules/fetch/FetchResponse.h</a></li>
<li><a href="#trunkSourceWebCoreplatformnetworkBlobResourceHandlecpp">trunk/Source/WebCore/platform/network/BlobResourceHandle.cpp</a></li>
</ul>
<h3>Removed Paths</h3>
<ul>
<li>trunk/LayoutTests/http/tests/fetch/</li>
<li><a href="#trunkLayoutTestshttptestsresourcesdownloadjsonwithdelayphp">trunk/LayoutTests/http/tests/resources/download-json-with-delay.php</a></li>
<li>trunk/LayoutTests/platform/gtk/imported/w3c/</li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkLayoutTestsChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/ChangeLog (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/ChangeLog 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/ChangeLog 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,3 +1,16 @@
</span><ins>+2016-03-24 Commit Queue <commit-queue@webkit.org>
+
+ Unreviewed, rolling out r198627.
+ https://bugs.webkit.org/show_bug.cgi?id=155856
+
+ Caused use-after-free (Requested by ap on #webkit).
+
+ Reverted changeset:
+
+ "[Fetch API] Add basic loading of resources"
+ https://bugs.webkit.org/show_bug.cgi?id=155637
+ http://trac.webkit.org/changeset/198627
+
</ins><span class="cx"> 2016-03-24 Saam barati <sbarati@apple.com>
</span><span class="cx">
</span><span class="cx"> Web Inspector: Separate Debugger enable state from the debugger breakpoints enabled state
</span></span></pre></div>
<a id="trunkLayoutTestsTestExpectations"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/TestExpectations (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/TestExpectations 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/TestExpectations 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -317,6 +317,10 @@
</span><span class="cx"> imported/w3c/web-platform-tests/XMLHttpRequest/send-redirect-to-cors.htm [ Skip ]
</span><span class="cx"> imported/w3c/web-platform-tests/XMLHttpRequest/send-redirect-to-non-cors.htm [ Skip ]
</span><span class="cx">
</span><ins>+# Tests that are flaky as failing assertions print a new token generated for each test run.
+imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob.html [ Failure ]
+imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob-worker.html [ Failure ]
+
</ins><span class="cx"> # New W3C ref tests that are failing.
</span><span class="cx"> webkit.org/b/148856 imported/w3c/web-platform-tests/html/semantics/embedded-content/the-video-element/video_initially_paused.html [ ImageOnlyFailure ]
</span><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestshttptestsresourcesdownloadjsonwithdelayphp"></a>
<div class="delfile"><h4>Deleted: trunk/LayoutTests/http/tests/resources/download-json-with-delay.php (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/http/tests/resources/download-json-with-delay.php 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/http/tests/resources/download-json-with-delay.php 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,20 +0,0 @@
</span><del>-<?php
-header("Expires: Thu, 01 Dec 2003 16:00:00 GMT");
-header("Cache-Control: no-cache, no-store, must-revalidate");
-header("Pragma: no-cache");
-header("Content-Type: application/x-no-buffering-please");
-
-$iteration = $_GET['iteration'];
-$delay = $_GET['delay'];
-
-echo "[$iteration, $delay ";
-
-for ($i = 1; $i <= $iteration; ++$i) {
- echo ", $i, \"foobar\"";
- // Force content to be sent to the browser as is.
- ob_flush();
- flush();
- usleep($delay * 1000);
-}
-echo "]";
-?>
</del></span></pre></div>
<a id="trunkLayoutTestsimportedw3cChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/ChangeLog (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/ChangeLog 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/ChangeLog 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,3 +1,16 @@
</span><ins>+2016-03-24 Commit Queue <commit-queue@webkit.org>
+
+ Unreviewed, rolling out r198627.
+ https://bugs.webkit.org/show_bug.cgi?id=155856
+
+ Caused use-after-free (Requested by ap on #webkit).
+
+ Reverted changeset:
+
+ "[Fetch API] Add basic loading of resources"
+ https://bugs.webkit.org/show_bug.cgi?id=155637
+ http://trac.webkit.org/changeset/198627
+
</ins><span class="cx"> 2016-03-24 Youenn Fablet <youenn.fablet@crf.canon.fr>
</span><span class="cx">
</span><span class="cx"> [Fetch API] Add basic loading of resources
</span></span></pre></div>
<a id="trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicacceptheaderexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/accept-header-expected.txt (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/accept-header-expected.txt 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/accept-header-expected.txt 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,3 +1,3 @@
</span><span class="cx">
</span><del>-PASS Request through fetch should have 'accept' header with value '*/*'
</del><ins>+FAIL Request through fetch should have 'accept' header with value '*/*' promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
</ins><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicintegrityexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/integrity-expected.txt (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/integrity-expected.txt 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/integrity-expected.txt 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,15 +1,15 @@
</span><span class="cx">
</span><del>-PASS Empty string integrity
-PASS SHA-256 integrity
-PASS SHA-384 integrity
-PASS SHA-512 integrity
-FAIL Invalid integrity assert_unreached: Should have rejected. Reached unreachable code
-PASS Multiple integrities: valid stronger than invalid
-FAIL Multiple integrities: invalid stronger than valid assert_unreached: Should have rejected. Reached unreachable code
-PASS Multiple integrities: invalid as strong as valid
-PASS Multiple integrities: both are valid
-FAIL Multiple integrities: both are invalid assert_unreached: Should have rejected. Reached unreachable code
-FAIL CORS empty integrity promise_test: Unhandled rejection with value: object "TypeError: Type error"
-FAIL CORS SHA-512 integrity promise_test: Unhandled rejection with value: object "TypeError: Type error"
-PASS CORS invalid integrity
</del><ins>+FAIL Empty string integrity promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL SHA-256 integrity promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL SHA-384 integrity promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL SHA-512 integrity promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Invalid integrity assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Multiple integrities: valid stronger than invalid promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Multiple integrities: invalid stronger than valid assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Multiple integrities: invalid as strong as valid promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Multiple integrities: both are valid promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Multiple integrities: both are invalid assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL CORS empty integrity promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL CORS SHA-512 integrity promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL CORS invalid integrity assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
</ins><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicmodenocorsexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/mode-no-cors-expected.txt (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/mode-no-cors-expected.txt 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/mode-no-cors-expected.txt 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,6 +1,6 @@
</span><span class="cx">
</span><del>-PASS Fetch ../resources/top.txt with no-cors mode
-PASS Fetch http://localhost:8800/fetch/api/resources/top.txt with no-cors mode
-FAIL Fetch https://localhost:9443/fetch/api/resources/top.txt with no-cors mode promise_test: Unhandled rejection with value: object "TypeError: Type error"
-FAIL Fetch http://www.localhost:8800/fetch/api/resources/top.txt with no-cors mode promise_test: Unhandled rejection with value: object "TypeError: Type error"
</del><ins>+FAIL Fetch ../resources/top.txt with no-cors mode promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetch http://localhost:8800/fetch/api/resources/top.txt with no-cors mode promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetch https://localhost:9443/fetch/api/resources/top.txt with no-cors mode promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetch http://www.localhost:8800/fetch/api/resources/top.txt with no-cors mode promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
</ins><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicmodesameoriginexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/mode-same-origin-expected.txt (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/mode-same-origin-expected.txt 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/mode-same-origin-expected.txt 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,6 +1,6 @@
</span><span class="cx">
</span><del>-PASS Fetch ../resources/top.txt with same-origin mode
-PASS Fetch http://localhost:8800/fetch/api/resources/top.txt with same-origin mode
-PASS Fetch https://localhost:9443/fetch/api/resources/top.txt with same-origin mode
-PASS Fetch http://www.localhost:8800/fetch/api/resources/top.txt with same-origin mode
</del><ins>+FAIL Fetch ../resources/top.txt with same-origin mode promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetch http://localhost:8800/fetch/api/resources/top.txt with same-origin mode promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetch https://localhost:9443/fetch/api/resources/top.txt with same-origin mode assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetch http://www.localhost:8800/fetch/api/resources/top.txt with same-origin mode assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
</ins><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicrequestforbiddenheadersexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/request-forbidden-headers-expected.txt (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/request-forbidden-headers-expected.txt 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/request-forbidden-headers-expected.txt 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,26 +1,26 @@
</span><span class="cx">
</span><del>-PASS Accept-Charset is a forbidden request header
-PASS Accept-Encoding is a forbidden request header
-PASS Access-Control-Request-Headers is a forbidden request header
-PASS Access-Control-Request-Method is a forbidden request header
-PASS Connection is a forbidden request header
-PASS Content-Length is a forbidden request header
-PASS Cookie is a forbidden request header
-PASS Cookie2 is a forbidden request header
-PASS Date is a forbidden request header
-PASS DNT is a forbidden request header
-PASS Expect is a forbidden request header
-PASS Host is a forbidden request header
-PASS Keep-Alive is a forbidden request header
-PASS Origin is a forbidden request header
-PASS Referer is a forbidden request header
-PASS TE is a forbidden request header
-PASS Trailer is a forbidden request header
-PASS Transfer-Encoding is a forbidden request header
-PASS Upgrade is a forbidden request header
-PASS Via is a forbidden request header
-PASS Proxy- is a forbidden request header
-PASS Proxy-Test is a forbidden request header
-PASS Sec- is a forbidden request header
-PASS Sec-Test is a forbidden request header
</del><ins>+FAIL Accept-Charset is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Accept-Encoding is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Access-Control-Request-Headers is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Access-Control-Request-Method is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Connection is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Content-Length is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Cookie is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Cookie2 is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Date is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL DNT is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Expect is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Host is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Keep-Alive is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Origin is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Referer is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL TE is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Trailer is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Transfer-Encoding is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Upgrade is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Via is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Proxy- is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Proxy-Test is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Sec- is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Sec-Test is a forbidden request header promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
</ins><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicrequestheadersexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/request-headers-expected.txt (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/request-headers-expected.txt 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/request-headers-expected.txt 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,11 +1,11 @@
</span><span class="cx">
</span><del>-FAIL Fetch with GET assert_equals: Request has header origin: http://localhost:8800 expected (string) "http://localhost:8800" but got (object) null
-FAIL Fetch with HEAD assert_equals: Request has header origin: http://localhost:8800 expected (string) "http://localhost:8800" but got (object) null
-FAIL Fetch with HEAD with body promise_test: Unhandled rejection with value: object "TypeError: Type error"
-PASS Fetch with PUT without body
-PASS Fetch with PUT with body
-PASS Fetch with POST without body
-PASS Fetch with POST with body
-FAIL Fetch with Chicken assert_equals: Request has header content-length: null expected (object) null but got (string) "0"
-PASS Fetch with Chicken with body
</del><ins>+FAIL Fetch with GET promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetch with HEAD promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetch with HEAD with body promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetch with PUT without body promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetch with PUT with body promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetch with POST without body promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetch with POST with body promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetch with Chicken promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetch with Chicken with body promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
</ins><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicschemeaboutexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-about-expected.txt (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-about-expected.txt 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-about-expected.txt 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,6 +1,6 @@
</span><span class="cx">
</span><del>-FAIL Fetching about:blank is OK promise_test: Unhandled rejection with value: object "TypeError: Type error"
-FAIL Fetching about:unicorn is OK promise_test: Unhandled rejection with value: object "TypeError: Type error"
-PASS Fetching about:invalid.com is KO
-PASS Fetching about:config is KO
</del><ins>+FAIL Fetching about:blank is OK promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetching about:unicorn is OK promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetching about:invalid.com is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching about:config is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
</ins><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicschemeblobexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob-expected.txt (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob-expected.txt 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob-expected.txt 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,5 +1,5 @@
</span><span class="cx">
</span><del>-PASS Regular Blob loading
-PASS Loading an erroneous blob scheme URL
-FAIL Loading a blob URL using POST assert_unreached: Should have rejected. Reached unreachable code
</del><ins>+FAIL Fetching blob:http://localhost:8800/4580501e-3c0f-4b03[...] is OK promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetching [GET] blob:http://www.localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching [POST] blob:http://localhost:8800/ad829fe3-9045-4b29[...] is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
</ins><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicschemeblobworkerexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob-worker-expected.txt (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob-worker-expected.txt 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob-worker-expected.txt 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,5 +1,5 @@
</span><span class="cx">
</span><del>-FAIL Regular Blob loading promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
-FAIL Loading an erroneous blob scheme URL assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
-FAIL Loading a blob URL using POST assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
</del><ins>+FAIL Fetching blob:http://localhost:8800/8807dca8-91bf-4414[...] is OK promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetching [GET] blob:http://www.localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching [POST] blob:http://localhost:8800/41805be3-d8cc-490f[...] is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
</ins><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicschemeblobjs"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob.js (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob.js 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-blob.js 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -4,6 +4,9 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> function checkFetchResponse(url, data, mime, size, desc) {
</span><ins>+ if (!desc)
+ var cut = (url.length >= 45) ? "[...]" : "";
+ desc = "Fetching " + url.substring(0, 45) + cut + " is OK"
</ins><span class="cx"> promise_test(function(test) {
</span><span class="cx"> size = size.toString();
</span><span class="cx"> return fetch(url).then(function(resp) {
</span><span class="lines">@@ -19,9 +22,12 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> var blob = new Blob(["Blob's data"], { "type" : "text/plain" });
</span><del>-checkFetchResponse(URL.createObjectURL(blob), "Blob's data", "text/plain", blob.size, "Regular Blob loading");
</del><ins>+checkFetchResponse(URL.createObjectURL(blob), "Blob's data", "text/plain", blob.size);
</ins><span class="cx">
</span><span class="cx"> function checkKoUrl(url, method, desc) {
</span><ins>+ if (!desc)
+ var cut = (url.length >= 45) ? "[...]" : "";
+ desc = "Fetching [" + method + "] " + url.substring(0, 45) + cut + " is KO"
</ins><span class="cx"> promise_test(function(test) {
</span><span class="cx"> var promise = fetch(url, {"method": method});
</span><span class="cx"> return promise_rejects(test, new TypeError(), promise);
</span><span class="lines">@@ -29,7 +35,7 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> var blob2 = new Blob(["Blob's data"], { "type" : "text/plain" });
</span><del>-checkKoUrl("blob:http://{{domains[www]}}:{{ports[http][0]}}/", "GET", "Loading an erroneous blob scheme URL");
-checkKoUrl(URL.createObjectURL(blob2), "POST", "Loading a blob URL using POST");
</del><ins>+checkKoUrl("blob:http://{{domains[www]}}:{{ports[http][0]}}/", "GET");
+checkKoUrl(URL.createObjectURL(blob2), "POST");
</ins><span class="cx">
</span><span class="cx"> done();
</span></span></pre></div>
<a id="trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicschemedataexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-data-expected.txt (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-data-expected.txt 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-data-expected.txt 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,8 +1,8 @@
</span><span class="cx">
</span><del>-FAIL Fetching data:,response%27s%20body is OK promise_test: Unhandled rejection with value: object "TypeError: Type error"
-FAIL Fetching data:text/plain;base64,cmVzcG9uc2UncyBib[...] is OK promise_test: Unhandled rejection with value: object "TypeError: Type error"
-FAIL Fetching data:image/png;base64,cmVzcG9uc2UncyBib2[...] is OK promise_test: Unhandled rejection with value: object "TypeError: Type error"
-PASS Fetching [GET] data:notAdataUrl.com is KO
-PASS Fetching [POST] data:,response%27s%20body is KO
-PASS Fetching [HEAD] data:,response%27s%20body is KO
</del><ins>+FAIL Fetching data:,response%27s%20body is OK promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetching data:text/plain;base64,cmVzcG9uc2UncyBib[...] is OK promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetching data:image/png;base64,cmVzcG9uc2UncyBib2[...] is OK promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
+FAIL Fetching [GET] data:notAdataUrl.com is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching [POST] data:,response%27s%20body is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching [HEAD] data:,response%27s%20body is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
</ins><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicschemeothersexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-others-expected.txt (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-others-expected.txt 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/scheme-others-expected.txt 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,18 +1,18 @@
</span><span class="cx">
</span><del>-PASS Fetching aaa://localhost:8800/ is KO
-PASS Fetching cap://localhost:8800/ is KO
-PASS Fetching cid://localhost:8800/ is KO
-PASS Fetching dav://localhost:8800/ is KO
-PASS Fetching dict://localhost:8800/ is KO
-PASS Fetching dns://localhost:8800/ is KO
-PASS Fetching geo://localhost:8800/ is KO
-PASS Fetching im://localhost:8800/ is KO
-PASS Fetching imap://localhost:8800/ is KO
-PASS Fetching ipp://localhost:8800/ is KO
-PASS Fetching ldap://localhost:8800/ is KO
-PASS Fetching mailto://localhost:8800/ is KO
-PASS Fetching nfs://localhost:8800/ is KO
-PASS Fetching pop://localhost:8800/ is KO
-PASS Fetching rtsp://localhost:8800/ is KO
-PASS Fetching snmp://localhost:8800/ is KO
</del><ins>+FAIL Fetching aaa://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching cap://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching cid://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching dav://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching dict://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching dns://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching geo://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching im://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching imap://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching ipp://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching ldap://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching mailto://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching nfs://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching pop://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching rtsp://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
+FAIL Fetching snmp://localhost:8800/ is KO assert_throws: function "function () { throw e }" threw "Fetch is not yet implemented" (undefined) expected object "TypeError" ("TypeError")
</ins><span class="cx">
</span></span></pre></div>
<a id="trunkLayoutTestsimportedw3cwebplatformtestsfetchapibasicstreamresponseexpectedtxt"></a>
<div class="modfile"><h4>Modified: trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/stream-response-expected.txt (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/stream-response-expected.txt 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/LayoutTests/imported/w3c/web-platform-tests/fetch/api/basic/stream-response-expected.txt 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,3 +1,3 @@
</span><span class="cx">
</span><del>-FAIL Stream response's body assert_unreached: Body does not exist in response Reached unreachable code
</del><ins>+FAIL Stream response's body promise_test: Unhandled rejection with value: "Fetch is not yet implemented"
</ins><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCoreChangeLog"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/ChangeLog (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/ChangeLog 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/ChangeLog 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -1,3 +1,16 @@
</span><ins>+2016-03-24 Commit Queue <commit-queue@webkit.org>
+
+ Unreviewed, rolling out r198627.
+ https://bugs.webkit.org/show_bug.cgi?id=155856
+
+ Caused use-after-free (Requested by ap on #webkit).
+
+ Reverted changeset:
+
+ "[Fetch API] Add basic loading of resources"
+ https://bugs.webkit.org/show_bug.cgi?id=155637
+ http://trac.webkit.org/changeset/198627
+
</ins><span class="cx"> 2016-03-24 Saam barati <sbarati@apple.com>
</span><span class="cx">
</span><span class="cx"> Web Inspector: Separate Debugger enable state from the debugger breakpoints enabled state
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchDOMWindowFetchcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/DOMWindowFetch.cpp (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/DOMWindowFetch.cpp 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/DOMWindowFetch.cpp 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -31,42 +31,16 @@
</span><span class="cx">
</span><span class="cx"> #if ENABLE(FETCH_API)
</span><span class="cx">
</span><del>-#include "DOMWindow.h"
-#include "FetchRequest.h"
-#include "FetchResponse.h"
-
</del><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><del>-void DOMWindowFetch::fetch(DOMWindow& window, FetchRequest* input, const Dictionary& dictionary, DeferredWrapper&& promise)
</del><ins>+void DOMWindowFetch::fetch(DOMWindow&, FetchRequest*, const Dictionary&, FetchPromise&& promise)
</ins><span class="cx"> {
</span><del>- if (!window.scriptExecutionContext())
- return;
- ScriptExecutionContext& context = *window.scriptExecutionContext();
-
- ExceptionCode ec = 0;
- RefPtr<FetchRequest> fetchRequest = FetchRequest::create(context, input, dictionary, ec);
- if (ec) {
- promise.reject(ec);
- return;
- }
- ASSERT(fetchRequest);
- FetchResponse::fetch(context, *fetchRequest, WTFMove(promise));
</del><ins>+ promise.reject(ASCIILiteral("Fetch is not yet implemented"));
</ins><span class="cx"> }
</span><span class="cx">
</span><del>-void DOMWindowFetch::fetch(DOMWindow& window, const String& url, const Dictionary& dictionary, DeferredWrapper&& promise)
</del><ins>+void DOMWindowFetch::fetch(DOMWindow&, const String&, const Dictionary&, FetchPromise&& promise)
</ins><span class="cx"> {
</span><del>- if (!window.scriptExecutionContext())
- return;
- ScriptExecutionContext& context = *window.scriptExecutionContext();
-
- ExceptionCode ec = 0;
- RefPtr<FetchRequest> fetchRequest = FetchRequest::create(context, url, dictionary, ec);
- if (ec) {
- promise.reject(ec);
- return;
- }
- ASSERT(fetchRequest);
- FetchResponse::fetch(context, *fetchRequest, WTFMove(promise));
</del><ins>+ promise.reject(ASCIILiteral("Fetch is not yet implemented"));
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> } // namespace WebCore
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchDOMWindowFetchh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/DOMWindowFetch.h (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/DOMWindowFetch.h 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/DOMWindowFetch.h 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -30,19 +30,19 @@
</span><span class="cx">
</span><span class="cx"> #if ENABLE(FETCH_API)
</span><span class="cx">
</span><del>-#include <wtf/Forward.h>
</del><ins>+#include "JSDOMPromise.h"
</ins><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><del>-class DOMWindow;
-class DeferredWrapper;
</del><span class="cx"> class Dictionary;
</span><span class="cx"> class FetchRequest;
</span><ins>+class FetchResponse;
</ins><span class="cx">
</span><span class="cx"> class DOMWindowFetch {
</span><span class="cx"> public:
</span><del>- static void fetch(DOMWindow&, FetchRequest*, const Dictionary&, DeferredWrapper&&);
- static void fetch(DOMWindow&, const String&, const Dictionary&, DeferredWrapper&&);
</del><ins>+ using FetchPromise = DOMPromise<RefPtr<FetchResponse>, String>;
+ static void fetch(DOMWindow&, FetchRequest*, const Dictionary&, FetchPromise&&);
+ static void fetch(DOMWindow&, const String&, const Dictionary&, FetchPromise&&);
</ins><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> } // namespace WebCore
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchFetchBodycpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/FetchBody.cpp (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/FetchBody.cpp 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/FetchBody.cpp 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -35,15 +35,12 @@
</span><span class="cx"> #include "Dictionary.h"
</span><span class="cx"> #include "ExceptionCode.h"
</span><span class="cx"> #include "FetchBodyOwner.h"
</span><del>-#include "FormData.h"
</del><span class="cx"> #include "HTTPParsers.h"
</span><span class="cx"> #include "JSBlob.h"
</span><span class="cx"> #include "JSDOMFormData.h"
</span><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><del>-static RefPtr<Blob> blobFromArrayBuffer(ArrayBuffer*, const String&);
-
</del><span class="cx"> FetchBody::FetchBody(Ref<Blob>&& blob)
</span><span class="cx"> : m_type(Type::Blob)
</span><span class="cx"> , m_mimeType(blob->type())
</span><span class="lines">@@ -158,46 +155,21 @@
</span><span class="cx">
</span><span class="cx"> void FetchBody::consume(FetchBodyOwner& owner, Consumer::Type type, DeferredWrapper&& promise)
</span><span class="cx"> {
</span><del>- if (m_type == Type::ArrayBuffer) {
- consumeArrayBuffer(type, promise);
- return;
- }
</del><span class="cx"> if (m_type == Type::Text) {
</span><del>- consumeText(type, promise);
</del><ins>+ consumeText(type, WTFMove(promise));
</ins><span class="cx"> return;
</span><span class="cx"> }
</span><span class="cx"> if (m_type == Type::Blob) {
</span><span class="cx"> consumeBlob(owner, type, WTFMove(promise));
</span><span class="cx"> return;
</span><span class="cx"> }
</span><del>- if (m_type == Type::Loading) {
- // FIXME: We should be able to change the loading type to text if consumer type is JSON or Text.
- m_consumer = Consumer({type, WTFMove(promise)});
- return;
- }
</del><span class="cx">
</span><span class="cx"> // FIXME: Support other types.
</span><span class="cx"> promise.reject<ExceptionCode>(0);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-void FetchBody::consumeArrayBuffer(Consumer::Type type, DeferredWrapper& promise)
</del><ins>+void FetchBody::consumeText(Consumer::Type type, DeferredWrapper&& promise)
</ins><span class="cx"> {
</span><del>- if (type == Consumer::Type::ArrayBuffer) {
- fulfillPromiseWithArrayBuffer(promise, m_data.get());
- return;
- }
- if (type == Consumer::Type::Blob) {
- promise.resolve(blobFromArrayBuffer(m_data.get(), Blob::normalizedContentType(extractMIMETypeFromMediaType(m_mimeType))));
- return;
- }
-
- ASSERT(type == Consumer::Type::Text || type == Consumer::Type::JSON);
- // FIXME: Do we need TextResourceDecoder to create a String to decode UTF-8 data.
- fulfillTextPromise(type, TextResourceDecoder::create(ASCIILiteral("text/plain"), "UTF-8")->decodeAndFlush(static_cast<const char*>(m_data->data()), m_data->byteLength()), promise);
-}
-
-void FetchBody::consumeText(Consumer::Type type, DeferredWrapper& promise)
-{
</del><span class="cx"> ASSERT(type == Consumer::Type::ArrayBuffer || type == Consumer::Type::Blob);
</span><span class="cx">
</span><span class="cx"> if (type == Consumer::Type::ArrayBuffer) {
</span><span class="lines">@@ -242,26 +214,6 @@
</span><span class="cx"> return value;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-static inline RefPtr<Blob> blobFromArrayBuffer(ArrayBuffer* buffer, const String& contentType)
-{
- if (!buffer)
- return Blob::create(Vector<char>(), contentType);
-
- // FIXME: We should try to move buffer to Blob without doing this copy.
- Vector<char> value(buffer->byteLength());
- memcpy(value.data(), buffer->data(), buffer->byteLength());
- return Blob::create(WTFMove(value), contentType);
-}
-
-void FetchBody::fulfillTextPromise(FetchBody::Consumer::Type type, const String& text, DeferredWrapper& promise)
-{
- ASSERT(type == Consumer::Type::Text || type == Consumer::Type::JSON);
- if (type == FetchBody::Consumer::Type::Text)
- promise.resolve(text);
- else
- fulfillPromiseWithJSON(promise, text);
-}
-
</del><span class="cx"> void FetchBody::loadingFailed()
</span><span class="cx"> {
</span><span class="cx"> ASSERT(m_consumer);
</span><span class="lines">@@ -271,23 +223,16 @@
</span><span class="cx">
</span><span class="cx"> void FetchBody::loadedAsArrayBuffer(RefPtr<ArrayBuffer>&& buffer)
</span><span class="cx"> {
</span><del>- if (m_type == Type::Loading) {
- m_type = Type::ArrayBuffer;
- m_data = buffer;
- if (m_consumer) {
- consumeArrayBuffer(m_consumer->type, m_consumer->promise);
- m_consumer = Nullopt;
- }
- return;
- }
-
</del><span class="cx"> ASSERT(m_consumer);
</span><span class="cx"> ASSERT(m_consumer->type == Consumer::Type::Blob || m_consumer->type == Consumer::Type::ArrayBuffer);
</span><span class="cx"> if (m_consumer->type == Consumer::Type::ArrayBuffer)
</span><span class="cx"> fulfillPromiseWithArrayBuffer(m_consumer->promise, buffer.get());
</span><span class="cx"> else {
</span><span class="cx"> ASSERT(m_blob);
</span><del>- m_consumer->promise.resolve(blobFromArrayBuffer(buffer.get(), m_blob->type()));
</del><ins>+ Vector<char> data;
+ data.reserveCapacity(buffer->byteLength());
+ data.append(static_cast<const char*>(buffer->data()), buffer->byteLength());
+ m_consumer->promise.resolve<RefPtr<Blob>>(Blob::create(WTFMove(data), m_blob->type()));
</ins><span class="cx"> }
</span><span class="cx"> m_consumer = Nullopt;
</span><span class="cx"> }
</span><span class="lines">@@ -296,26 +241,13 @@
</span><span class="cx"> {
</span><span class="cx"> ASSERT(m_consumer);
</span><span class="cx"> ASSERT(m_consumer->type == Consumer::Type::Text || m_consumer->type == Consumer::Type::JSON);
</span><del>-
- fulfillTextPromise(m_consumer->type, text, m_consumer->promise);
</del><ins>+ if (m_consumer->type == Consumer::Type::Text)
+ m_consumer->promise.resolve(text);
+ else
+ fulfillPromiseWithJSON(m_consumer->promise, text);
</ins><span class="cx"> m_consumer = Nullopt;
</span><span class="cx"> }
</span><span class="cx">
</span><del>-RefPtr<FormData> FetchBody::bodyForInternalRequest() const
-{
- if (m_type == Type::None)
- return nullptr;
- if (m_type == Type::Text)
- return FormData::create(UTF8Encoding().encode(m_text, EntitiesForUnencodables));
- if (m_type == Type::Blob) {
- RefPtr<FormData> body = FormData::create();
- body->appendBlob(m_blob->url());
- return body;
- }
- ASSERT_NOT_REACHED();
- return nullptr;
</del><span class="cx"> }
</span><span class="cx">
</span><del>-}
-
</del><span class="cx"> #endif // ENABLE(FETCH_API)
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchFetchBodyh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/FetchBody.h (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/FetchBody.h 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/FetchBody.h 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -44,7 +44,6 @@
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><span class="cx"> class FetchBodyOwner;
</span><del>-class FormData;
</del><span class="cx">
</span><span class="cx"> class FetchBody {
</span><span class="cx"> public:
</span><span class="lines">@@ -62,22 +61,18 @@
</span><span class="cx">
</span><span class="cx"> static FetchBody extract(JSC::ExecState&, JSC::JSValue);
</span><span class="cx"> static FetchBody extractFromBody(FetchBody*);
</span><del>- static FetchBody loadingBody() { return { Type::Loading }; }
</del><span class="cx"> FetchBody() = default;
</span><span class="cx">
</span><span class="cx"> void loadingFailed();
</span><span class="cx"> void loadedAsArrayBuffer(RefPtr<ArrayBuffer>&&);
</span><span class="cx"> void loadedAsText(String&&);
</span><span class="cx">
</span><del>- RefPtr<FormData> bodyForInternalRequest() const;
-
</del><span class="cx"> private:
</span><del>- enum class Type { None, ArrayBuffer, Loading, Text, Blob, FormData };
</del><ins>+ enum class Type { None, Text, Blob, FormData };
</ins><span class="cx">
</span><span class="cx"> FetchBody(Ref<Blob>&&);
</span><span class="cx"> FetchBody(Ref<DOMFormData>&&);
</span><span class="cx"> FetchBody(String&&);
</span><del>- FetchBody(Type type) : m_type(type) { }
</del><span class="cx">
</span><span class="cx"> struct Consumer {
</span><span class="cx"> enum class Type { Text, Blob, JSON, ArrayBuffer };
</span><span class="lines">@@ -89,21 +84,17 @@
</span><span class="cx">
</span><span class="cx"> Vector<char> extractFromText() const;
</span><span class="cx"> bool processIfEmptyOrDisturbed(Consumer::Type, DeferredWrapper&);
</span><del>- void consumeArrayBuffer(Consumer::Type, DeferredWrapper&);
- void consumeText(Consumer::Type, DeferredWrapper&);
</del><ins>+ void consumeText(Consumer::Type, DeferredWrapper&&);
</ins><span class="cx"> void consumeBlob(FetchBodyOwner&, Consumer::Type, DeferredWrapper&&);
</span><span class="cx"> static FetchLoader::Type loadingType(Consumer::Type);
</span><del>- static void fulfillTextPromise(FetchBody::Consumer::Type, const String&, DeferredWrapper&);
- static void fulfillArrayBufferPromise(FetchBody::Consumer::Type, const String&, DeferredWrapper&);
</del><span class="cx">
</span><del>- Type m_type { Type::None };
</del><ins>+ Type m_type = Type::None;
</ins><span class="cx"> String m_mimeType;
</span><del>- bool m_isDisturbed { false };
</del><ins>+ bool m_isDisturbed = false;
</ins><span class="cx">
</span><span class="cx"> // FIXME: Add support for BufferSource and URLSearchParams.
</span><span class="cx"> RefPtr<Blob> m_blob;
</span><span class="cx"> RefPtr<DOMFormData> m_formData;
</span><del>- RefPtr<ArrayBuffer> m_data;
</del><span class="cx"> String m_text;
</span><span class="cx">
</span><span class="cx"> Optional<Consumer> m_consumer;
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchFetchBodyOwnercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/FetchBodyOwner.cpp (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/FetchBodyOwner.cpp 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/FetchBodyOwner.cpp 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -59,16 +59,19 @@
</span><span class="cx"> ASSERT(m_body.isDisturbed());
</span><span class="cx"> ASSERT(!m_blobLoader);
</span><span class="cx">
</span><del>- if (!scriptExecutionContext())
- blobLoadingFailed();
-
</del><span class="cx"> m_blobLoader = { *this };
</span><span class="cx"> m_blobLoader->loader = std::make_unique<FetchLoader>(type, *m_blobLoader);
</span><span class="cx">
</span><span class="cx"> setPendingActivity(this);
</span><del>- m_blobLoader->loader->start(*scriptExecutionContext(), blob);
</del><ins>+ if (!scriptExecutionContext() || !m_blobLoader->loader->start(*scriptExecutionContext(), blob))
+ blobLoadingFailed();
</ins><span class="cx"> }
</span><span class="cx">
</span><ins>+void FetchBodyOwner::loadedBlobAsText(String&& text)
+{
+ m_body.loadedAsText(WTFMove(text));
+}
+
</ins><span class="cx"> void FetchBodyOwner::finishBlobLoading()
</span><span class="cx"> {
</span><span class="cx"> ASSERT(m_blobLoader);
</span><span class="lines">@@ -77,11 +80,6 @@
</span><span class="cx"> unsetPendingActivity(this);
</span><span class="cx"> }
</span><span class="cx">
</span><del>-void FetchBodyOwner::loadedBlobAsText(String&& text)
-{
- m_body.loadedAsText(WTFMove(text));
-}
-
</del><span class="cx"> void FetchBodyOwner::blobLoadingFailed()
</span><span class="cx"> {
</span><span class="cx"> m_body.loadingFailed();
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchFetchBodyOwnerh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/FetchBodyOwner.h (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/FetchBodyOwner.h 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/FetchBodyOwner.h 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -54,13 +54,6 @@
</span><span class="cx">
</span><span class="cx"> bool isActive() const { return !!m_blobLoader; }
</span><span class="cx">
</span><del>-protected:
- const FetchBody& body() const { return m_body; }
- FetchBody& body() { return m_body; }
-
- // ActiveDOMObject API
- void stop() override;
-
</del><span class="cx"> private:
</span><span class="cx"> // Blob loading routines
</span><span class="cx"> void loadedBlobAsText(String&&);
</span><span class="lines">@@ -69,6 +62,9 @@
</span><span class="cx"> void blobLoadingFailed();
</span><span class="cx"> void finishBlobLoading();
</span><span class="cx">
</span><ins>+ // ActiveDOMObject API
+ void stop() override;
+
</ins><span class="cx"> struct BlobLoader final : FetchLoaderClient {
</span><span class="cx"> BlobLoader(FetchBodyOwner&);
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchFetchHeaderscpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/FetchHeaders.cpp (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/FetchHeaders.cpp 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/FetchHeaders.cpp 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -156,19 +156,14 @@
</span><span class="cx">
</span><span class="cx"> void FetchHeaders::fill(const FetchHeaders* headers)
</span><span class="cx"> {
</span><del>- ASSERT(m_guard != Guard::Immutable);
-
</del><span class="cx"> if (!headers)
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- filterAndFill(headers->m_headers, m_guard);
-}
</del><ins>+ ASSERT(m_guard != Guard::Immutable);
</ins><span class="cx">
</span><del>-void FetchHeaders::filterAndFill(const HTTPHeaderMap& headers, Guard guard)
-{
</del><span class="cx"> ExceptionCode ec;
</span><del>- for (auto& header : headers) {
- if (canWriteHeader(header.key, header.value, guard, ec)) {
</del><ins>+ for (auto& header : headers->m_headers) {
+ if (canWriteHeader(header.key, header.value, m_guard, ec)) {
</ins><span class="cx"> if (header.keyAsHTTPHeaderName)
</span><span class="cx"> m_headers.add(header.keyAsHTTPHeaderName.value(), header.value);
</span><span class="cx"> else
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchFetchHeadersh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/FetchHeaders.h (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/FetchHeaders.h 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/FetchHeaders.h 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -64,8 +64,6 @@
</span><span class="cx">
</span><span class="cx"> void fill(const FetchHeaders*);
</span><span class="cx">
</span><del>- void filterAndFill(const HTTPHeaderMap&, Guard);
-
</del><span class="cx"> String fastGet(HTTPHeaderName name) const { return m_headers.get(name); }
</span><span class="cx"> void fastSet(HTTPHeaderName name, const String& value) { m_headers.set(name, value); }
</span><span class="cx">
</span><span class="lines">@@ -81,8 +79,6 @@
</span><span class="cx"> };
</span><span class="cx"> Iterator createIterator() { return Iterator(*this); }
</span><span class="cx">
</span><del>- const HTTPHeaderMap& internalHeaders() const { return m_headers; }
-
</del><span class="cx"> private:
</span><span class="cx"> FetchHeaders(Guard guard) : m_guard(guard) { }
</span><span class="cx"> FetchHeaders(Guard guard, const HTTPHeaderMap& headers) : m_guard(guard), m_headers(headers) { }
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchFetchLoadercpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/FetchLoader.cpp (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/FetchLoader.cpp 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/FetchLoader.cpp 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -34,10 +34,8 @@
</span><span class="cx"> #include "BlobURL.h"
</span><span class="cx"> #include "FetchBody.h"
</span><span class="cx"> #include "FetchLoaderClient.h"
</span><del>-#include "FetchRequest.h"
</del><span class="cx"> #include "ResourceRequest.h"
</span><span class="cx"> #include "ScriptExecutionContext.h"
</span><del>-#include "SecurityOrigin.h"
</del><span class="cx"> #include "SharedBuffer.h"
</span><span class="cx"> #include "TextResourceDecoder.h"
</span><span class="cx"> #include "ThreadableBlobRegistry.h"
</span><span class="lines">@@ -45,14 +43,11 @@
</span><span class="cx">
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><del>-void FetchLoader::start(ScriptExecutionContext& context, Blob& blob)
</del><ins>+bool FetchLoader::start(ScriptExecutionContext& context, Blob& blob)
</ins><span class="cx"> {
</span><span class="cx"> auto urlForReading = BlobURL::createPublicURL(context.securityOrigin());
</span><del>- if (urlForReading.isEmpty()) {
- m_client.didFail();
- return;
- }
-
</del><ins>+ if (urlForReading.isEmpty())
+ return false;
</ins><span class="cx"> ThreadableBlobRegistry::registerBlobURL(context.securityOrigin(), urlForReading, blob.url());
</span><span class="cx">
</span><span class="cx"> ResourceRequest request(urlForReading);
</span><span class="lines">@@ -68,23 +63,9 @@
</span><span class="cx"> options.contentSecurityPolicyEnforcement = ContentSecurityPolicyEnforcement::DoNotEnforce;
</span><span class="cx">
</span><span class="cx"> m_loader = ThreadableLoader::create(&context, this, request, options);
</span><ins>+ return true;
</ins><span class="cx"> }
</span><span class="cx">
</span><del>-void FetchLoader::start(ScriptExecutionContext& context, const FetchRequest& request)
-{
- // FIXME: Compute loading options according fetch options.
- ThreadableLoaderOptions options;
- options.setSendLoadCallbacks(SendCallbacks);
- options.setSniffContent(DoNotSniffContent);
- options.setDataBufferingPolicy(DoNotBufferData);
- options.preflightPolicy = ConsiderPreflight;
- options.setAllowCredentials(AllowStoredCredentials);
- options.crossOriginRequestPolicy = DenyCrossOriginRequests;
- options.contentSecurityPolicyEnforcement = ContentSecurityPolicyEnforcement::DoNotEnforce;
-
- m_loader = ThreadableLoader::create(&context, this, request.internalRequest(), options);
-}
-
</del><span class="cx"> FetchLoader::FetchLoader(Type type, FetchLoaderClient& client)
</span><span class="cx"> : m_type(type)
</span><span class="cx"> , m_client(client)
</span><span class="lines">@@ -132,11 +113,6 @@
</span><span class="cx"> m_client.didFail();
</span><span class="cx"> }
</span><span class="cx">
</span><del>-void FetchLoader::didFailRedirectCheck()
-{
- m_client.didFail();
-}
-
</del><span class="cx"> } // namespace WebCore
</span><span class="cx">
</span><span class="cx"> #endif // ENABLE(FETCH_API)
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchFetchLoaderh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/FetchLoader.h (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/FetchLoader.h 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/FetchLoader.h 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -39,7 +39,6 @@
</span><span class="cx">
</span><span class="cx"> class Blob;
</span><span class="cx"> class FetchLoaderClient;
</span><del>-class FetchRequest;
</del><span class="cx"> class ScriptExecutionContext;
</span><span class="cx">
</span><span class="cx"> class FetchLoader final : public ThreadableLoaderClient {
</span><span class="lines">@@ -48,8 +47,7 @@
</span><span class="cx">
</span><span class="cx"> FetchLoader(Type, FetchLoaderClient&);
</span><span class="cx">
</span><del>- void start(ScriptExecutionContext&, const FetchRequest&);
- void start(ScriptExecutionContext&, Blob&);
</del><ins>+ bool start(ScriptExecutionContext&, Blob&);
</ins><span class="cx"> void stop();
</span><span class="cx">
</span><span class="cx"> private:
</span><span class="lines">@@ -58,7 +56,6 @@
</span><span class="cx"> void didReceiveData(const char*, int) final;
</span><span class="cx"> void didFinishLoading(unsigned long, double) final;
</span><span class="cx"> void didFail(const ResourceError&) final;
</span><del>- void didFailRedirectCheck() final;
</del><span class="cx">
</span><span class="cx"> private:
</span><span class="cx"> Type m_type { Type::ArrayBuffer };
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchFetchRequestcpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/FetchRequest.cpp (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/FetchRequest.cpp 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/FetchRequest.cpp 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -435,14 +435,6 @@
</span><span class="cx"> return String();
</span><span class="cx"> }
</span><span class="cx">
</span><del>-ResourceRequest FetchRequest::internalRequest() const
-{
- ResourceRequest request = m_internalRequest.request;
- request.setHTTPHeaderFields(m_headers->internalHeaders());
- request.setHTTPBody(body().bodyForInternalRequest());
- return request;
-}
-
</del><span class="cx"> RefPtr<FetchRequest> FetchRequest::clone(ScriptExecutionContext& context, ExceptionCode& ec)
</span><span class="cx"> {
</span><span class="cx"> if (isDisturbed()) {
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchFetchRequesth"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/FetchRequest.h (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/FetchRequest.h 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/FetchRequest.h 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -72,9 +72,6 @@
</span><span class="cx"> String integrity;
</span><span class="cx"> };
</span><span class="cx">
</span><del>- const FetchOptions& fetchOptions() const { return m_internalRequest.options; }
- ResourceRequest internalRequest() const;
-
</del><span class="cx"> private:
</span><span class="cx"> FetchRequest(ScriptExecutionContext&, FetchBody&&, Ref<FetchHeaders>&&, InternalRequest&&);
</span><span class="cx">
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchFetchResponsecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/FetchResponse.cpp (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/FetchResponse.cpp 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/FetchResponse.cpp 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -153,77 +153,6 @@
</span><span class="cx"> return JSC::jsNull();
</span><span class="cx"> }
</span><span class="cx">
</span><del>-void FetchResponse::fetch(ScriptExecutionContext& context, const FetchRequest& request, FetchPromise&& promise)
-{
- Ref<FetchResponse> response = adoptRef(*new FetchResponse(context, Type::Basic, FetchBody::loadingBody(), FetchHeaders::create(FetchHeaders::Guard::Immutable), ResourceResponse()));
-
- // Setting pending activity until BodyLoader didFail or didSucceed callback is called.
- response->setPendingActivity(response.ptr());
-
- response->m_bodyLoader = BodyLoader(response.get(), WTFMove(promise));
- response->m_bodyLoader->start(context, request);
-}
-
-void FetchResponse::BodyLoader::didSucceed()
-{
- m_response.m_bodyLoader = Nullopt;
- m_response.unsetPendingActivity(&m_response);
-}
-
-void FetchResponse::BodyLoader::didFail()
-{
- if (m_promise)
- std::exchange(m_promise, Nullopt)->reject(TypeError);
-
- m_response.m_bodyLoader = Nullopt;
- m_response.unsetPendingActivity(&m_response);
-
- // FIXME: Handle the case of failing after didReceiveResponse is called.
-}
-
-FetchResponse::BodyLoader::BodyLoader(FetchResponse& response, FetchPromise&& promise)
- : m_response(response)
- , m_promise(WTFMove(promise))
-{
-}
-
-void FetchResponse::BodyLoader::didReceiveResponse(const ResourceResponse& resourceResponse)
-{
- ASSERT(m_promise);
-
- m_response.m_response = resourceResponse;
- m_response.m_headers->filterAndFill(resourceResponse.httpHeaderFields(), FetchHeaders::Guard::Response);
-
- std::exchange(m_promise, Nullopt)->resolve(&m_response);
-}
-
-void FetchResponse::BodyLoader::didFinishLoadingAsArrayBuffer(RefPtr<ArrayBuffer>&& buffer)
-{
- m_response.body().loadedAsArrayBuffer(WTFMove(buffer));
-}
-
-void FetchResponse::BodyLoader::start(ScriptExecutionContext& context, const FetchRequest& request)
-{
- m_loader = std::make_unique<FetchLoader>(FetchLoader::Type::ArrayBuffer, *this);
- m_loader->start(context, request);
-}
-
-void FetchResponse::BodyLoader::stop()
-{
- if (m_loader)
- m_loader->stop();
-}
-
-void FetchResponse::stop()
-{
- FetchBodyOwner::stop();
- if (m_bodyLoader) {
- RefPtr<FetchResponse> protect(this);
- m_bodyLoader->stop();
- m_bodyLoader = Nullopt;
- }
-}
-
</del><span class="cx"> const char* FetchResponse::activeDOMObjectName() const
</span><span class="cx"> {
</span><span class="cx"> return "Response";
</span></span></pre></div>
<a id="trunkSourceWebCoreModulesfetchFetchResponseh"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/Modules/fetch/FetchResponse.h (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/Modules/fetch/FetchResponse.h 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/Modules/fetch/FetchResponse.h 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -42,7 +42,6 @@
</span><span class="cx"> namespace WebCore {
</span><span class="cx">
</span><span class="cx"> class Dictionary;
</span><del>-class FetchRequest;
</del><span class="cx">
</span><span class="cx"> typedef int ExceptionCode;
</span><span class="cx">
</span><span class="lines">@@ -54,9 +53,6 @@
</span><span class="cx"> // FIXME: Binding generator should not require below method to handle optional status parameter.
</span><span class="cx"> static RefPtr<FetchResponse> redirect(ScriptExecutionContext& context, const String& url, ExceptionCode& ec) { return redirect(context, url, 302, ec); }
</span><span class="cx">
</span><del>- using FetchPromise = DOMPromise<RefPtr<FetchResponse>, ExceptionCode>;
- static void fetch(ScriptExecutionContext&, const FetchRequest&, FetchPromise&&);
-
</del><span class="cx"> void initializeWith(const Dictionary&, ExceptionCode&);
</span><span class="cx">
</span><span class="cx"> String type() const;
</span><span class="lines">@@ -75,35 +71,14 @@
</span><span class="cx"> FetchResponse(ScriptExecutionContext&, Type, FetchBody&&, Ref<FetchHeaders>&&, ResourceResponse&&);
</span><span class="cx">
</span><span class="cx"> // ActiveDOMObject API
</span><del>- void stop() final;
</del><span class="cx"> const char* activeDOMObjectName() const final;
</span><span class="cx"> bool canSuspendForDocumentSuspension() const final;
</span><span class="cx">
</span><del>- class BodyLoader final : public FetchLoaderClient {
- public:
- BodyLoader(FetchResponse&, FetchPromise&&);
-
- void start(ScriptExecutionContext&, const FetchRequest&);
- void stop();
-
- private:
- // FetchLoaderClient API
- void didSucceed() final;
- void didFail() final;
- void didReceiveResponse(const ResourceResponse&);
- void didFinishLoadingAsArrayBuffer(RefPtr<ArrayBuffer>&&) final;
-
- FetchResponse& m_response;
- Optional<FetchPromise> m_promise;
- std::unique_ptr<FetchLoader> m_loader;
- };
-
</del><span class="cx"> Type m_type;
</span><span class="cx"> ResourceResponse m_response;
</span><span class="cx"> Ref<FetchHeaders> m_headers;
</span><span class="cx"> bool m_isLocked = false;
</span><span class="cx"> bool m_isRedirected = false;
</span><del>- Optional<BodyLoader> m_bodyLoader;
</del><span class="cx"> };
</span><span class="cx">
</span><span class="cx"> } // namespace WebCore
</span></span></pre></div>
<a id="trunkSourceWebCoreplatformnetworkBlobResourceHandlecpp"></a>
<div class="modfile"><h4>Modified: trunk/Source/WebCore/platform/network/BlobResourceHandle.cpp (198648 => 198649)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/Source/WebCore/platform/network/BlobResourceHandle.cpp 2016-03-24 22:43:06 UTC (rev 198648)
+++ trunk/Source/WebCore/platform/network/BlobResourceHandle.cpp 2016-03-24 23:01:45 UTC (rev 198649)
</span><span class="lines">@@ -136,6 +136,10 @@
</span><span class="cx">
</span><span class="cx"> PassRefPtr<BlobResourceHandle> BlobResourceHandle::createAsync(BlobData* blobData, const ResourceRequest& request, ResourceHandleClient* client)
</span><span class="cx"> {
</span><ins>+ // FIXME: Should probably call didFail() instead of blocking the load without explanation.
+ if (!equalLettersIgnoringASCIICase(request.httpMethod(), "get"))
+ return nullptr;
+
</ins><span class="cx"> return adoptRef(new BlobResourceHandle(blobData, request, client, true));
</span><span class="cx"> }
</span><span class="cx">
</span><span class="lines">@@ -203,12 +207,6 @@
</span><span class="cx"> if (m_aborted || m_errorCode)
</span><span class="cx"> return;
</span><span class="cx">
</span><del>- if (!equalLettersIgnoringASCIICase(firstRequest().httpMethod(), "get")) {
- m_errorCode = methodNotAllowed;
- notifyResponse();
- return;
- }
-
</del><span class="cx"> // If the blob data is not found, fail now.
</span><span class="cx"> if (!m_blobData) {
</span><span class="cx"> m_errorCode = notFoundError;
</span><span class="lines">@@ -580,10 +578,6 @@
</span><span class="cx"> ResourceResponse response(firstRequest().url(), m_blobData->contentType(), m_totalRemainingSize, String());
</span><span class="cx"> response.setHTTPStatusCode(isRangeRequest ? httpPartialContent : httpOK);
</span><span class="cx"> response.setHTTPStatusText(isRangeRequest ? httpPartialContentText : httpOKText);
</span><del>-
- response.setHTTPHeaderField(HTTPHeaderName::ContentType, m_blobData->contentType());
- response.setHTTPHeaderField(HTTPHeaderName::ContentLength, String::number(m_totalRemainingSize));
-
</del><span class="cx"> if (isRangeRequest)
</span><span class="cx"> response.setHTTPHeaderField(HTTPHeaderName::ContentRange, ParsedContentRange(m_rangeOffset, m_rangeEnd, m_totalSize).headerValue());
</span><span class="cx"> // FIXME: If a resource identified with a blob: URL is a File object, user agents must use that file's name attribute,
</span></span></pre>
</div>
</div>
</body>
</html>