[ServiceWorker] Send the body of Request in fetch() API.

Source/modules/serviceworkers/FetchManager.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "FetchManager.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "bindings/core/v8/ScriptState.h"
@@ skipping 252 matching lines @@
     // FIXME: Support body.
     ResourceRequest request(m_request->url());
     request.setRequestContext(WebURLRequest::RequestContextFetch);
     request.setDownloadToFile(true);
     request.setHTTPMethod(m_request->method());
     const Vector<OwnPtr<FetchHeaderList::Header> >& list = m_request->headerList()->list();
     for (size_t i = 0; i < list.size(); ++i) {
         request.addHTTPHeaderField(AtomicString(list[i]->first), AtomicString(list[i]->second));
     }
 
+    if (m_request->method() != "GET" && m_request->method() != "HEAD") {

[michaeln1, 2014/09/03 20:03:44]

Would it be better to test positive for "POST" and "PUT" (so unexpected inputs
don't get a post body), or is there a reason you're testing negative?

Ok, i see XMLHttpRequest::areMethodAndURLValidForSend() tests negative like this
too, it also tests for m_url.protocolIsInHTTPFamily. Should this method test for
that as well?

[horo, 2014/09/04 05:21:01]

I want to do the same as XMLHttpRequest.
performHTTPFetch() is called only when m_request->url().protocolIsInHTTPFamily()
it true.
Added ASSERT() in the top of performHTTPFetch().

+        RefPtr<BlobDataHandle> blobDataHandle = m_request->blobDataHandle();
+        if (blobDataHandle.get()) {
+            RefPtr<FormData> httpBody(FormData::create());
+            httpBody->appendBlob(blobDataHandle->uuid(), blobDataHandle);
+            request.setHTTPBody(httpBody);
+        }
+    }
+
     // "2. Append `Referer`/empty byte sequence, if |HTTPRequest|'s |referrer|
     // is none, and `Referer`/|HTTPRequest|'s referrer, serialized and utf-8
     // encoded, otherwise, to HTTPRequest's header list.
     // We set the referrer using workerGlobalScope's URL in
     // WorkerThreadableLoader.
 
     // "3. Append `Host`, ..."
     // FIXME: Implement this when the spec is fixed.
 
     // "4.If |HTTPRequest|'s force Origin header flag is set, append `Origin`/
@@ skipping 63 matching lines @@
     loader->start();
     return promise;
 }
 
 void FetchManager::onLoaderFinished(Loader* loader)
 {
     m_loaders.remove(loader);
 }
 
 } // namespace blink