Fetch API: Stop lowercasing header names.

third_party/WebKit/Source/modules/cachestorage/Cache.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/cachestorage/Cache.h"
 
 #include "bindings/core/v8/CallbackPromiseAdapter.h"
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "bindings/core/v8/ScriptState.h"
@@ skipping 187 matching lines @@
   }
   DEFINE_THREAD_SAFE_STATIC_LOCAL(
       EnumerationHistogram, response_type_histogram,
       new EnumerationHistogram("ServiceWorkerCache.Cache.AddResponseType",
                                static_cast<int>(ResponseType::kEnumMax)));
   response_type_histogram.Count(static_cast<int>(type));
 };
 
 bool VaryHeaderContainsAsterisk(const Response* response) {
   const FetchHeaderList* headers = response->headers()->HeaderList();
-  for (size_t i = 0; i < headers->size(); ++i) {
-    const FetchHeaderList::Header& header = headers->Entry(i);
-    if (header.first == "vary") {
+  for (const auto& header : headers->List()) {
+    if (EqualIgnoringASCIICase(header.first, "vary")) {
       Vector<String> fields;
       header.second.Split(',', fields);
       for (size_t j = 0; j < fields.size(); ++j) {

[nhiroki, 2017/04/12 06:53:05]

Can you replace this with range-based-for?

         if (fields[j].StripWhiteSpace() == "*")
           return true;
       }
     }
   }
   return false;
 }
 
 }  // namespace
 
@@ skipping 477 matching lines @@
       WTF::MakeUnique<CacheWithRequestsCallbacks>(resolver), web_request,
       ToWebQueryParams(options));
   return promise;
 }
 
 WebServiceWorkerCache* Cache::WebCache() const {
   return web_cache_.get();
 }
 
 }  // namespace blink