[ServiceWorker] Use BodyStreamBuffer as the body data of the Response object.

Source/modules/serviceworkers/FetchRequestData.cpp

Index: Source/modules/serviceworkers/FetchRequestData.cpp
diff --git a/Source/modules/serviceworkers/FetchRequestData.cpp b/Source/modules/serviceworkers/FetchRequestData.cpp
index 6d214f15362cf1cc26f84d4bf32ffbdc8b3a9067..23a136ef82bca405dcc9f7a69d3df9bec105f342 100644
--- a/Source/modules/serviceworkers/FetchRequestData.cpp
+++ b/Source/modules/serviceworkers/FetchRequestData.cpp
@@ -68,6 +68,13 @@ FetchRequestData::FetchRequestData()
 {
 }
 
+String FetchRequestData::contentType() const

[yhirano, 2014/12/12 12:42:33]

Is this function's meaning clear?
This function is called only in Body, when m_blobDataHandle is null. It means
that this function always return an empty string and that seems wrong.

[horo, 2014/12/15 02:12:24]

Removed and I introduced contentTypeForBuffer to Body.

+{
+    if (m_blobDataHandle)
+        return m_blobDataHandle->type();
+    return "";

[yhirano, 2014/12/12 12:42:32]

return String() is more efficient.

[horo, 2014/12/15 02:12:24]

Done in Request::contentTypeForBuffer().

+}
+
 void FetchRequestData::trace(Visitor* visitor)
 {
     visitor->trace(m_headerList);