Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(489)

Unified Diff: third_party/WebKit/Source/modules/cachestorage/CacheTest.cpp

Issue 2810843002: bindings: Make the sequence conversion code more complaint with WebIDL. (Closed)
Patch Set: Adjust even more tests Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/modules/cachestorage/CacheTest.cpp
diff --git a/third_party/WebKit/Source/modules/cachestorage/CacheTest.cpp b/third_party/WebKit/Source/modules/cachestorage/CacheTest.cpp
index 7759069320f6ddd685cc3a251813da98e99b7ed4..863210c8834c512d58c374abbbe4929883ac8bc1 100644
--- a/third_party/WebKit/Source/modules/cachestorage/CacheTest.cpp
+++ b/third_party/WebKit/Source/modules/cachestorage/CacheTest.cpp
@@ -8,6 +8,8 @@
#include <memory>
#include <string>
#include "bindings/core/v8/ExceptionState.h"
+#include "bindings/core/v8/IDLTypes.h"
+#include "bindings/core/v8/NativeValueTraitsImpl.h"
#include "bindings/core/v8/ScriptFunction.h"
#include "bindings/core/v8/ScriptPromise.h"
#include "bindings/core/v8/ScriptPromiseResolver.h"
@@ -611,14 +613,13 @@ TEST_F(CacheStorageTest, KeysResponseTest) {
ScriptPromise result = cache->keys(GetScriptState(), exception_state);
ScriptValue script_value = GetResolveValue(result);
- Vector<v8::Local<v8::Value>> requests =
- ToImplArray<Vector<v8::Local<v8::Value>>>(script_value.V8Value(), 0,
- GetIsolate(), exception_state);
+ HeapVector<Member<Request>> requests =
+ NativeValueTraits<IDLSequence<Request>>::NativeValue(
+ GetIsolate(), script_value.V8Value(), exception_state);
EXPECT_EQ(expected_urls.size(), requests.size());
for (int i = 0, minsize = std::min(expected_urls.size(), requests.size());
i < minsize; ++i) {
- Request* request =
- V8Request::toImplWithTypeCheck(GetIsolate(), requests[i]);
+ Request* request = requests[i];
EXPECT_TRUE(request);
if (request)
EXPECT_EQ(expected_urls[i], request->url());
@@ -674,14 +675,13 @@ TEST_F(CacheStorageTest, MatchAllAndBatchResponseTest) {
options, exception_state);
ScriptValue script_value = GetResolveValue(result);
- Vector<v8::Local<v8::Value>> responses =
- ToImplArray<Vector<v8::Local<v8::Value>>>(script_value.V8Value(), 0,
- GetIsolate(), exception_state);
+ HeapVector<Member<Response>> responses =
+ NativeValueTraits<IDLSequence<Response>>::NativeValue(
+ GetIsolate(), script_value.V8Value(), exception_state);
EXPECT_EQ(expected_urls.size(), responses.size());
for (int i = 0, minsize = std::min(expected_urls.size(), responses.size());
i < minsize; ++i) {
- Response* response =
- V8Response::toImplWithTypeCheck(GetIsolate(), responses[i]);
+ Response* response = responses[i];
EXPECT_TRUE(response);
if (response)
EXPECT_EQ(expected_urls[i], response->url());

Powered by Google App Engine
This is Rietveld 408576698