Fetch API: Stop lowercasing header names.

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 "core/dom/Iterator.h"
 #include "platform/loader/fetch/FetchUtils.h"
-#include "platform/wtf/NotFound.h"
-#include "platform/wtf/PassRefPtr.h"
-#include "platform/wtf/RefPtr.h"
 #include "platform/wtf/text/WTFString.h"
 
 namespace blink {
 
 namespace {
 
 class HeadersIterationSource final
     : public PairIterable<String, String>::IterationSource {
  public:
   explicit HeadersIterationSource(const FetchHeaderList* headers)
-      : headers_(headers->Clone()), current_(0) {
-    headers_->SortAndCombine();
-  }
+      : headers_(headers->SortAndCombine()), current_(0) {}
 
   bool Next(ScriptState* script_state,
             String& key,
             String& value,
             ExceptionState& exception) override {
     // This simply advances an index and returns the next value if any; the
     // iterated list is not exposed to script so it will never be mutated
     // during iteration.
-    if (current_ >= headers_->size())
+    if (current_ >= headers_.size())
       return false;
 
-    const FetchHeaderList::Header& header = headers_->Entry(current_++);
+    const FetchHeaderList::Header& header = headers_.at(current_++);
     key = header.first;
     value = header.second;
     return true;
   }
 
   DEFINE_INLINE_VIRTUAL_TRACE() {
-    visitor->Trace(headers_);
     PairIterable<String, String>::IterationSource::Trace(visitor);
   }
 
  private:
-  const Member<FetchHeaderList> headers_;
+  Vector<std::pair<String, String>> headers_;
   size_t current_;
 };
 
 }  // namespace
 
 Headers* Headers::Create(ExceptionState&) {
   return new Headers;
 }
 
 Headers* Headers::Create(
@@ skipping 173 matching lines @@
     return;
   // "6. Set |name|/|value| in header list."
   header_list_->Set(name, value);
 }
 
 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 (size_t i = 0; i < object->header_list_->List().size(); ++i) {
-    append(object->header_list_->List()[i]->first,
-           object->header_list_->List()[i]->second, exception_state);
+  for (const auto& header : object->header_list_->List()) {
+    append(header.first, header.second, exception_state);
     if (exception_state.HadException())
       return;
   }
 }
 
 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