bindings: Make the sequence conversion code more complaint with WebIDL.

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 <memory>
+#include <utility>
 #include "bindings/core/v8/CallbackPromiseAdapter.h"
 #include "bindings/core/v8/ExceptionState.h"
+#include "bindings/core/v8/IDLTypes.h"
+#include "bindings/core/v8/NativeValueTraitsImpl.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "bindings/core/v8/ScriptState.h"
 #include "bindings/core/v8/V8Binding.h"
 #include "bindings/core/v8/V8ThrowException.h"
 #include "bindings/modules/v8/V8Response.h"
 #include "core/dom/DOMException.h"
 #include "core/inspector/ConsoleMessage.h"
 #include "modules/cachestorage/CacheStorageError.h"
 #include "modules/fetch/BodyStreamBuffer.h"
 #include "modules/fetch/FetchDataLoader.h"
 #include "modules/fetch/GlobalFetch.h"
 #include "modules/fetch/Request.h"
 #include "modules/fetch/Response.h"
 #include "platform/HTTPNames.h"
 #include "platform/Histogram.h"
 #include "public/platform/modules/serviceworker/WebServiceWorkerCache.h"
-#include <memory>
-#include <utility>
 
 namespace blink {
 
 namespace {
 
 // FIXME: Consider using CallbackPromiseAdapter.
 class CacheMatchCallbacks : public WebServiceWorkerCache::CacheMatchCallbacks {
   WTF_MAKE_NONCOPYABLE(CacheMatchCallbacks);
 
  public:
@@ skipping 192 matching lines @@
       ScriptState* script_state,
       Cache* cache,
       const HeapVector<Member<Request>>& requests) {
     FetchResolvedForAdd* self =
         new FetchResolvedForAdd(script_state, cache, requests);
     return self->BindToV8Function();
   }
 
   ScriptValue Call(ScriptValue value) override {
     NonThrowableExceptionState exception_state;
-    HeapVector<Member<Response>> responses = ToMemberNativeArray<Response>(
-        value.V8Value(), requests_.size(), GetScriptState()->GetIsolate(),
-        exception_state);
+    HeapVector<Member<Response>> responses =
+        NativeValueTraits<IDLSequence<Response>>::NativeValue(
+            GetScriptState()->GetIsolate(), value.V8Value(), exception_state);
 
     for (const auto& response : responses) {
       if (!response->ok()) {
         ScriptPromise rejection = ScriptPromise::Reject(
             GetScriptState(),
             V8ThrowException::CreateTypeError(GetScriptState()->GetIsolate(),
                                               "Request failed"));
         return ScriptValue(GetScriptState(), rejection.V8Value());
       }
       if (VaryHeaderContainsAsterisk(response)) {
@@ skipping 449 matching lines @@
       WTF::MakeUnique<CacheWithRequestsCallbacks>(resolver), web_request,
       ToWebQueryParams(options));
   return promise;
 }
 
 WebServiceWorkerCache* Cache::WebCache() const {
   return web_cache_.get();
 }
 
 }  // namespace blink