ServiceWorker: support Response.{status,statusText,headers} [blink] (2/3)

Source/platform/exported/WebServiceWorkerResponse.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 "public/platform/WebServiceWorkerResponse.h"
 
 namespace blink {
 
-void WebServiceWorkerResponse::setStatusCode(unsigned short statusCode)
+class WebServiceWorkerResponsePrivate : public RefCounted<WebServiceWorkerResponsePrivate> {

[falken, 2014/05/26 09:06:59]

I have a naive question: what's the motivation for using this class instead of
just having the private fields directly in WebServiceWorkerResponse? Do we
expect multiple WebServiceWorkerResponse to refer to the same Private? (I see
it's a common pattern in public...)

[kinuko, 2014/05/26 13:51:35]

We want to store map-like data in WebServiceWorkerResponse but we don't have
public exportable type at public blink interface for that (like WebVector for
Vector), so I moved them into private impl class. (The refcnt'ed shared part is
not really necessary in our case, I just chose it for easier copy handling)

[falken, 2014/05/26 14:59:31]

Ah I see!

+public:
+    unsigned short status;
+    WebString statusText;
+    HashMap<String, String> headers;
+};
+
+WebServiceWorkerResponse::WebServiceWorkerResponse()
+    : m_private(adoptRef(new WebServiceWorkerResponsePrivate))
 {
-    m_statusCode = statusCode;
 }
 
-unsigned short WebServiceWorkerResponse::statusCode() const
+void WebServiceWorkerResponse::reset()
 {
-    return m_statusCode;
+    m_private.reset();
+}
+
+void WebServiceWorkerResponse::assign(const WebServiceWorkerResponse& other)
+{
+    m_private = other.m_private;
+}
+
+void WebServiceWorkerResponse::setStatus(unsigned short status)
+{
+    m_private->status = status;
+}
+
+unsigned short WebServiceWorkerResponse::status() const
+{
+    return m_private->status;
 }
 
 void WebServiceWorkerResponse::setStatusText(const WebString& statusText)
 {
-    m_statusText = statusText;
+    m_private->statusText = statusText;
 }
 
 WebString WebServiceWorkerResponse::statusText() const
 {
-    return m_statusText;
+    return m_private->statusText;
 }
 
-void WebServiceWorkerResponse::setMethod(const WebString& method)
+void WebServiceWorkerResponse::setHeader(const WebString& key, const WebString& value)
 {
-    m_method = method;
+    m_private->headers.set(key, value);
 }
 
-WebString WebServiceWorkerResponse::method() const
+WebVector<WebString> WebServiceWorkerResponse::getHeaderKeys() const
 {
-    return m_method;
+    Vector<String> keys;
+    copyKeysToVector(m_private->headers, keys);
+    return keys;
+}
+
+WebString WebServiceWorkerResponse::getHeader(const WebString& key) const
+{
+    return m_private->headers.get(key);
+}
+
+void WebServiceWorkerResponse::setHeaders(const HashMap<String, String>& headers)
+{
+    m_private->headers = headers;
+}
+
+const HashMap<String, String>& WebServiceWorkerResponse::headers() const
+{
+    return m_private->headers;
 }
 
 } // namespace blink