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

Source/modules/serviceworkers/FetchManager.cpp

Index: Source/modules/serviceworkers/FetchManager.cpp
diff --git a/Source/modules/serviceworkers/FetchManager.cpp b/Source/modules/serviceworkers/FetchManager.cpp
index bbb06e183f3cedeadfcc13c679a19aae053b1db0..d09433732fe609d9b804a3e16f596a74849e6754 100644
--- a/Source/modules/serviceworkers/FetchManager.cpp
+++ b/Source/modules/serviceworkers/FetchManager.cpp
@@ -270,6 +270,15 @@ void FetchManager::Loader::performHTTPFetch()
         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.