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

Source/modules/serviceworkers/Response.idl

 // 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.
 
 [
     Constructor(optional Dictionary responseInitDict),
     RuntimeEnabled=ServiceWorker,
     Exposed=ServiceWorker
 ] interface Response {
-    attribute unsigned short statusCode;
+    attribute unsigned short status;
 
-    // FIXME: Spec uses ByteString for these. We must perform the DOMString -> ByteString conversion manually (crbug.com/347426).
+    // FIXME: Spec uses ByteString for this. We must perform the DOMString -> ByteString conversion manually (crbug.com/347426).
     attribute DOMString statusText;
-    attribute DOMString method;
 
-    // FIXME: Implement HeaderMap which uses the MapClass WebIDL attribute (crbug.com/347435).
-    // FIXME: Can't return Dictionaries yet (crbug.com/364025).
-    // getter Dictionary headers();
-    setter void headers(Dictionary items);
+    // Implements https://github.com/slightlyoff/ServiceWorker/issues/268
+    attribute HeaderMap headers;
+    getter DOMString getHeader(DOMString value);

[jsbell, 2014/05/23 20:59:49]

Looks like the getter/setter are gone in the latest spec?

[kinuko, 2014/05/26 05:45:13]

Yes, so I updated the code.

+    setter void setHeader(DOMString name, DOMString value);
 
     // FIXME: Implement the following:
     // attribute DOMString url;
     // Promise<Blob> toBlob();
 };