[ServiceWorker] Make Request class better conformance with the spec.

Source/modules/serviceworkers/Headers.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 Headers_h
 #define Headers_h
 
 #include "bindings/v8/ScriptWrappable.h"
 #include "modules/serviceworkers/FetchHeaderList.h"
 #include "wtf/Forward.h"
@@ skipping 29 matching lines @@
     Vector<String> getAll(const String& key, ExceptionState&);
     bool has(const String& key, ExceptionState&);
     void set(const String& key, const String& value, ExceptionState&);
     unsigned long size() const;
     void forEach(PassOwnPtr<HeadersForEachCallback>, ScriptValue&);
     void forEach(PassOwnPtr<HeadersForEachCallback>);
 
     void setGuard(Guard guard) { m_guard = guard; }
     Guard guard() const { return m_guard; }
 
+    void fillWith(const Headers*, ExceptionState&);

[yhirano, 2014/07/02 01:53:07]

Please describe that |this| must be cleared before calling these methods.

[horo, 2014/07/02 03:33:04]

Done.

+    void fillWith(const Dictionary&, ExceptionState&);
+
 private:
     Headers();
     // Shares the FetchHeaderList. Called when creating a Request or Response.
     explicit Headers(FetchHeaderList*);
     void forEachInternal(PassOwnPtr<HeadersForEachCallback>, ScriptValue*);
 
-    void fillWith(const Headers*, ExceptionState&);
-    void fillWith(const Dictionary&, ExceptionState&);
-
     RefPtr<FetchHeaderList> m_headerList;
     Guard m_guard;
 };
 
 } // namespace WebCore
 
 #endif // Headers_h