ServiceWorker: Add a Response ctor that accepts the body as a Blob

Source/modules/serviceworkers/Response.cpp

Index: Source/modules/serviceworkers/Response.cpp
diff --git a/Source/modules/serviceworkers/Response.cpp b/Source/modules/serviceworkers/Response.cpp
index 9b4c9c1111f453ba1207b4b4e99458c70d57dcb5..f71da330a51c8b5ff6b9fe62bb2e34901f0fddb8 100644
--- a/Source/modules/serviceworkers/Response.cpp
+++ b/Source/modules/serviceworkers/Response.cpp
@@ -6,20 +6,23 @@
 #include "Response.h"
 
 #include "bindings/v8/Dictionary.h"
+#include "core/fileapi/Blob.h"
 #include "modules/serviceworkers/ResponseInit.h"
 #include "platform/NotImplemented.h"
 #include "public/platform/WebServiceWorkerResponse.h"
 
 namespace WebCore {
 
-PassRefPtr<Response> Response::create()
+// FIXME: Remove this legacy function when the required Chromium-side patch lands.
+PassRefPtr<Response> Response::create(const Dictionary& responseInit)
 {
-    return create(Dictionary());
+    return adoptRef(new Response("", ResponseInit(responseInit)));
 }
 
-PassRefPtr<Response> Response::create(const Dictionary& responseInit)
+PassRefPtr<Response> Response::create(Blob* body, const Dictionary& responseInit)
 {
-    return adoptRef(new Response(ResponseInit(responseInit)));
+    // FIXME: Maybe append or override content-length and content-type headers using the blob. The spec will clarify what to do.

[dominicc (has gone to gerrit), 2014/06/02 00:50:47]

Include a link to a specific issue if possible so it is easy for someone to tell
whether the FIXME has become fixable.

[falken, 2014/06/02 03:55:27]

Done.

+    return adoptRef(new Response(body->uuid(), ResponseInit(responseInit)));
 }
 
 PassRefPtr<HeaderMap> Response::headers() const
@@ -33,12 +36,14 @@ void Response::populateWebServiceWorkerResponse(blink::WebServiceWorkerResponse&
     response.setStatus(status());
     response.setStatusText(statusText());
     response.setHeaders(m_headers->headerMap());
+    response.setBlobUUID(m_blobUUID);
 }
 
-Response::Response(const ResponseInit& responseInit)
+Response::Response(const String& blobUUID, const ResponseInit& responseInit)
     : m_status(responseInit.status)
     , m_statusText(responseInit.statusText)
     , m_headers(responseInit.headers)
+    , m_blobUUID(blobUUID)
 {
     ScriptWrappable::init(this);