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

Source/modules/serviceworkers/Response.h

 // 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.
 
 #ifndef Response_h
 #define Response_h
 
 #include "bindings/v8/Dictionary.h"
 #include "bindings/v8/ScriptWrappable.h"
 #include "modules/serviceworkers/HeaderMap.h"
 #include "wtf/RefCounted.h"
 #include "wtf/RefPtr.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink { class WebServiceWorkerResponse; }
 
 namespace WebCore {
 
+class Blob;
 struct ResponseInit;
 
 class Response FINAL : public ScriptWrappable, public RefCounted<Response> {
 public:
-    static PassRefPtr<Response> create();
     static PassRefPtr<Response> create(const Dictionary& responseInit);
+    static PassRefPtr<Response> create(Blob* body, const Dictionary& responseInit);
     ~Response() { };
 
     unsigned short status() const { return m_status; }
     String statusText() const { return m_statusText; }
     PassRefPtr<HeaderMap> headers() const;
 
     void populateWebServiceWorkerResponse(blink::WebServiceWorkerResponse&);
 
 private:
-    explicit Response(const ResponseInit&);
+    Response(const String& blobUUID, const ResponseInit&);
     unsigned short m_status;
     String m_statusText;
     RefPtr<HeaderMap> m_headers;
+    String m_blobUUID;

[kinuko, 2014/05/30 12:53:15]

I think this should hold a reference to Blob (or, rather, to BlobDataHandle),
otherwise this uuid could become invalid before we dereference it?

[michaeln1, 2014/05/30 20:09:35]

yup, the ctor probably should take as input a ptr to the BDH

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

I see! Done.

 };
 
 } // namespace WebCore
 
 #endif // Response_h