fetch: Drop Headers from HeadersInit typedef declaration.

third_party/WebKit/Source/modules/fetch/Headers.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 "modules/fetch/Headers.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/V8IteratorResultValue.h"
-#include "bindings/modules/v8/ByteStringSequenceSequenceOrByteStringByteStringRecordOrHeaders.h"
+#include "bindings/modules/v8/ByteStringSequenceSequenceOrByteStringByteStringRecord.h"
 #include "core/dom/Iterator.h"
 #include "platform/loader/fetch/FetchUtils.h"
 #include "platform/wtf/text/WTFString.h"
 
 namespace blink {
 
 namespace {
 
 class HeadersIterationSource final
     : public PairIterable<String, String>::IterationSource {
@@ skipping 179 matching lines @@
     return;
   // "5. Otherwise, if guard is |response| and |name| is a forbidden response
   //     header name, return."
   if (guard_ == kResponseGuard &&
       FetchUtils::IsForbiddenResponseHeaderName(name))
     return;
   // "6. Set |name|/|value| in header list."
   header_list_->Set(name, value);
 }
 
+// This overload is not called directly by Web APIs, but rather by other C++
+// classes. For example, when initializing a Request object it is possible that
+// a Request's Headers must be filled with an existing Headers object.
 void Headers::FillWith(const Headers* object, ExceptionState& exception_state) {
   DCHECK(header_list_->size() == 0);
-  // There used to be specific steps describing filling a Headers object with
-  // another Headers object, but it has since been removed because it should be
-  // handled like a sequence (http://crbug.com/690428).
   for (const auto& header : object->header_list_->List()) {
     append(header.first, header.second, exception_state);
     if (exception_state.HadException())
       return;
   }
 }
 
 void Headers::FillWith(const HeadersInit& init,
                        ExceptionState& exception_state) {
   DCHECK_EQ(header_list_->size(), 0U);
   if (init.isByteStringSequenceSequence()) {
     FillWith(init.getAsByteStringSequenceSequence(), exception_state);
   } else if (init.isByteStringByteStringRecord()) {
     FillWith(init.getAsByteStringByteStringRecord(), exception_state);
-  } else if (init.isHeaders()) {
-    // This branch will not be necessary once http://crbug.com/690428 is fixed.
-    FillWith(init.getAsHeaders(), exception_state);
   } else {
     NOTREACHED();
   }
 }
 
 void Headers::FillWith(const Vector<Vector<String>>& object,
                        ExceptionState& exception_state) {
   DCHECK(!header_list_->size());
   // "1. If |object| is a sequence, then for each |header| in |object|, run
   //     these substeps:
@@ skipping 33 matching lines @@
   visitor->Trace(header_list_);
 }
 
 PairIterable<String, String>::IterationSource* Headers::StartIteration(
     ScriptState*,
     ExceptionState&) {
   return new HeadersIterationSource(header_list_);
 }
 
 }  // namespace blink