Service Worker: Cache.put() consumes request/response bodies

Source/modules/serviceworkers/Response.cpp

 // 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.
 
 #include "config.h"
 #include "Response.h"
 
 #include "bindings/core/v8/Dictionary.h"
 #include "bindings/core/v8/ExceptionState.h"
 #include "core/dom/DOMArrayBuffer.h"
@@ skipping 221 matching lines @@
 
 Response* Response::clone() const
 {
     return Response::create(*this);
 }
 
 void Response::populateWebServiceWorkerResponse(WebServiceWorkerResponse& response)
 {
     m_response->populateWebServiceWorkerResponse(response);
 }
 

[gavinp, 2014/11/13 19:35:03]

Does Response::hasBody() go here?

(and Response::trace too?)

 Response::Response(ExecutionContext* context)
     : Body(context)
     , m_response(FetchResponseData::create())
     , m_headers(Headers::create(m_response->headerList()))
 {
     m_headers->setGuard(Headers::ResponseGuard);
 }
 
 Response::Response(const Response& copy_from)
     : Body(copy_from)
     , m_response(copy_from.m_response)
     , m_headers(copy_from.m_headers->createCopy())
 {
 }
 
 Response::Response(ExecutionContext* context, FetchResponseData* response)
     : Body(context)
     , m_response(response)
     , m_headers(Headers::create(m_response->headerList()))
 {
     m_headers->setGuard(Headers::ResponseGuard);
 }
 
 
 PassRefPtr<BlobDataHandle> Response::blobDataHandle()
 {
     return m_response->blobDataHandle();
 }
 
+bool Response::hasBody() const
+{
+    return m_response->blobDataHandle();
+}
+
 void Response::trace(Visitor* visitor)
 {
     Body::trace(visitor);
     visitor->trace(m_response);
     visitor->trace(m_headers);
 }
 
 } // namespace blink