[Fetch API] Put body members directly on Response/Request

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/core/v8/Dictionary.h"
 #include "bindings/core/v8/ScriptWrappable.h"
-#include "modules/serviceworkers/FetchBodyStream.h"
+#include "modules/serviceworkers/Body.h"
 #include "modules/serviceworkers/FetchResponseData.h"
 #include "modules/serviceworkers/Headers.h"
 #include "platform/blob/BlobData.h"
 #include "platform/heap/Handle.h"
 #include "wtf/RefPtr.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
 
 class Blob;
 class ExceptionState;
 class ResponseInit;
 class WebServiceWorkerResponse;
 
-class Response FINAL : public GarbageCollected<Response>, public ScriptWrappable {
+class Response FINAL : public Body {
     DEFINE_WRAPPERTYPEINFO();
 public:
-    static Response* create(Blob*, const Dictionary&, ExceptionState&);
-    static Response* create(const String&, const Dictionary&, ExceptionState&);
-    static Response* create(Blob*, const ResponseInit&, ExceptionState&);
-    static Response* create(FetchResponseData*);
-    static Response* create(const WebServiceWorkerResponse&);
+    virtual ~Response() { }
+    static Response* create(ExecutionContext*, Blob*, const Dictionary&, ExceptionState&);
+    static Response* create(ExecutionContext*, const String&, const Dictionary&, ExceptionState&);
+    static Response* create(ExecutionContext*, Blob*, const ResponseInit&, ExceptionState&);
+    static Response* create(ExecutionContext*, FetchResponseData*);
+    static Response* create(ExecutionContext*, const WebServiceWorkerResponse&);
 
     String type() const;
     String url() const;
     unsigned short status() const;
     String statusText() const;
     Headers* headers() const;
 
-    FetchBodyStream* body(ExecutionContext*);
-
     void populateWebServiceWorkerResponse(WebServiceWorkerResponse&);
 
-    void trace(Visitor*);
+    virtual void trace(Visitor*) OVERRIDE;
 
 private:
-    Response();
-    explicit Response(FetchResponseData*);
-    explicit Response(const WebServiceWorkerResponse&);
+    Response(ExecutionContext*);

[jsbell, 2014/09/09 22:04:25]

single argument constructor should have `explicit` qualifier

[horo, 2014/09/10 06:18:44]

Done.

+    Response(ExecutionContext*, FetchResponseData*);
+    Response(ExecutionContext*, const WebServiceWorkerResponse&);
+
+    virtual PassRefPtr<BlobDataHandle> blobDataHandle() OVERRIDE;
 
     Member<FetchResponseData> m_response;
     Member<Headers> m_headers;
-    Member<FetchBodyStream> m_fetchBodyStream;
 };
 
 } // namespace blink
 
 #endif // Response_h