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

Unified Diff: third_party/WebKit/Source/bindings/core/v8/NativeValueTraitsImplTest.cpp

Issue 2732093003: bindings: Add support for the record<K,V> WebIDL type. (Closed)
Patch Set: It turns out static was indeed necessary Created 3 years, 9 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/bindings/core/v8/NativeValueTraitsImplTest.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/NativeValueTraitsImplTest.cpp b/third_party/WebKit/Source/bindings/core/v8/NativeValueTraitsImplTest.cpp
new file mode 100644
index 0000000000000000000000000000000000000000..908de7431793b44179c2e82070b560fbdada65f3
--- /dev/null
+++ b/third_party/WebKit/Source/bindings/core/v8/NativeValueTraitsImplTest.cpp
@@ -0,0 +1,170 @@
+// Copyright 2017 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 "bindings/core/v8/NativeValueTraitsImpl.h"
+
+#include <utility>
+#include "bindings/core/v8/ExceptionState.h"
+#include "bindings/core/v8/IDLTypes.h"
+#include "bindings/core/v8/ToV8.h"
+#include "bindings/core/v8/V8BindingForTesting.h"
+#include "testing/gtest/include/gtest/gtest.h"
+#include "wtf/Vector.h"
+
+namespace blink {
+
+namespace {
+
+template <typename T>
+v8::Local<v8::Value> ToV8(V8TestingScope* scope, T value) {
+ return blink::ToV8(value, scope->context()->Global(), scope->isolate());
+}
+
+TEST(NativeValueTraitsImplTest, IDLRecord) {
+ V8TestingScope scope;
+ {
+ v8::Local<v8::Object> v8Object = v8::Object::New(scope.isolate());
+ NonThrowableExceptionState exceptionState;
+ const auto& record =
+ NativeValueTraits<IDLRecord<IDLString, IDLOctet>>::nativeValue(
+ scope.isolate(), v8Object, exceptionState);
+ EXPECT_TRUE(record.isEmpty());
+ }
+ {
+ v8::Local<v8::Object> v8Object = v8::Object::New(scope.isolate());
+ EXPECT_TRUE(v8CallBoolean(
+ v8Object->Set(scope.context(), ToV8(&scope, "foo"), ToV8(&scope, 34))));
+ EXPECT_TRUE(v8CallBoolean(v8Object->Set(
+ scope.context(), ToV8(&scope, "bar"), ToV8(&scope, -1024))));
+ EXPECT_TRUE(v8CallBoolean(
+ v8Object->Set(scope.context(), ToV8(&scope, "foo"), ToV8(&scope, 42))));
+
+ NonThrowableExceptionState exceptionState;
+ const auto& record =
+ NativeValueTraits<IDLRecord<IDLByteString, IDLLong>>::nativeValue(
+ scope.isolate(), v8Object, exceptionState);
+ EXPECT_EQ(2U, record.size());
+ EXPECT_EQ(std::make_pair(String("foo"), int32_t(42)), record[0]);
+ EXPECT_EQ(std::make_pair(String("bar"), int32_t(-1024)), record[1]);
+ }
+ {
+ v8::Local<v8::Object> v8Object = v8::Object::New(scope.isolate());
+ EXPECT_TRUE(v8CallBoolean(
+ v8Object->Set(scope.context(), ToV8(&scope, "foo"), ToV8(&scope, 42))));
+ EXPECT_TRUE(v8CallBoolean(
+ v8Object->Set(scope.context(), ToV8(&scope, "bar"), ToV8(&scope, 0))));
+ EXPECT_TRUE(v8CallBoolean(v8Object->Set(scope.context(), ToV8(&scope, "xx"),
+ ToV8(&scope, true))));
+ EXPECT_TRUE(v8CallBoolean(v8Object->Set(
+ scope.context(), ToV8(&scope, "abcd"), ToV8(&scope, false))));
+
+ NonThrowableExceptionState exceptionState;
+ const auto& record =
+ NativeValueTraits<IDLRecord<IDLByteString, IDLBoolean>>::nativeValue(
+ scope.isolate(), v8Object, exceptionState);
+ EXPECT_EQ(4U, record.size());
+ EXPECT_EQ(std::make_pair(String("foo"), true), record[0]);
+ EXPECT_EQ(std::make_pair(String("bar"), false), record[1]);
+ EXPECT_EQ(std::make_pair(String("xx"), true), record[2]);
+ EXPECT_EQ(std::make_pair(String("abcd"), false), record[3]);
+ }
+ {
+ v8::Local<v8::Array> v8StringArray = v8::Array::New(scope.isolate(), 2);
+ EXPECT_TRUE(v8CallBoolean(v8StringArray->Set(
+ scope.context(), ToV8(&scope, 0), ToV8(&scope, "Hello, World!"))));
+ EXPECT_TRUE(v8CallBoolean(v8StringArray->Set(
+ scope.context(), ToV8(&scope, 1), ToV8(&scope, "Hi, Mom!"))));
+ EXPECT_TRUE(v8CallBoolean(v8StringArray->Set(
+ scope.context(), ToV8(&scope, "foo"), ToV8(&scope, "Ohai"))));
+
+ NonThrowableExceptionState exceptionState;
+ const auto& record =
+ NativeValueTraits<IDLRecord<IDLUSVString, IDLString>>::nativeValue(
+ scope.isolate(), v8StringArray, exceptionState);
+ EXPECT_EQ(3U, record.size());
+ EXPECT_EQ(std::make_pair(String("0"), String("Hello, World!")), record[0]);
+ EXPECT_EQ(std::make_pair(String("1"), String("Hi, Mom!")), record[1]);
+ EXPECT_EQ(std::make_pair(String("foo"), String("Ohai")), record[2]);
+ }
+ {
+ v8::Local<v8::Object> v8Object = v8::Object::New(scope.isolate());
+ EXPECT_TRUE(v8CallBoolean(v8Object->Set(
+ scope.context(), v8::Symbol::GetToStringTag(scope.isolate()),
+ ToV8(&scope, 34))));
+ EXPECT_TRUE(v8CallBoolean(
+ v8Object->Set(scope.context(), ToV8(&scope, "foo"), ToV8(&scope, 42))));
+
+ NonThrowableExceptionState exceptionState;
+ const auto& record =
+ NativeValueTraits<IDLRecord<IDLString, IDLShort>>::nativeValue(
+ scope.isolate(), v8Object, exceptionState);
+ EXPECT_EQ(1U, record.size());
+ EXPECT_EQ(std::make_pair(String("foo"), int16_t(42)), record[0]);
+ }
+ {
+ v8::Local<v8::Object> v8Object = v8::Object::New(scope.isolate());
+ EXPECT_TRUE(v8CallBoolean(v8Object->Set(
+ scope.context(), v8::Symbol::GetToStringTag(scope.isolate()),
+ ToV8(&scope, 34))));
+ EXPECT_TRUE(v8CallBoolean(
+ v8Object->Set(scope.context(), ToV8(&scope, "foo"), ToV8(&scope, 42))));
+
+ NonThrowableExceptionState exceptionState;
+ const auto& record =
+ NativeValueTraits<IDLRecord<IDLString, IDLShort>>::nativeValue(
+ scope.isolate(), v8Object, exceptionState);
+ EXPECT_EQ(1U, record.size());
+ EXPECT_EQ(std::make_pair(String("foo"), int16_t(42)), record[0]);
+ }
+ {
+ v8::Local<v8::Object> v8ParentObject = v8::Object::New(scope.isolate());
+ EXPECT_TRUE(v8CallBoolean(v8ParentObject->Set(
+ scope.context(), ToV8(&scope, "foo"), ToV8(&scope, 34))));
+ EXPECT_TRUE(v8CallBoolean(v8ParentObject->Set(
+ scope.context(), ToV8(&scope, "bar"), ToV8(&scope, 512))));
+ v8::Local<v8::Object> v8Object = v8::Object::New(scope.isolate());
+ EXPECT_TRUE(
+ v8CallBoolean(v8Object->SetPrototype(scope.context(), v8ParentObject)));
+
+ NonThrowableExceptionState exceptionState;
+ auto record =
+ NativeValueTraits<IDLRecord<IDLString, IDLUnsignedLong>>::nativeValue(
+ scope.isolate(), v8Object, exceptionState);
+ EXPECT_TRUE(record.isEmpty());
+
+ EXPECT_TRUE(v8CallBoolean(v8Object->Set(
+ scope.context(), ToV8(&scope, "quux"), ToV8(&scope, 42))));
+ EXPECT_TRUE(v8CallBoolean(v8Object->Set(
+ scope.context(), ToV8(&scope, "foo"), ToV8(&scope, 1024))));
+ record =
+ NativeValueTraits<IDLRecord<IDLString, IDLUnsignedLong>>::nativeValue(
+ scope.isolate(), v8Object, exceptionState);
+ EXPECT_EQ(2U, record.size());
+ EXPECT_EQ(std::make_pair(String("quux"), uint32_t(42)), record[0]);
+ EXPECT_EQ(std::make_pair(String("foo"), uint32_t(1024)), record[1]);
+ }
+ {
+ v8::Local<v8::Array> v8StringArray = v8::Array::New(scope.isolate(), 2);
+ EXPECT_TRUE(v8CallBoolean(v8StringArray->Set(
+ scope.context(), ToV8(&scope, 0), ToV8(&scope, "Hello, World!"))));
+ EXPECT_TRUE(v8CallBoolean(v8StringArray->Set(
+ scope.context(), ToV8(&scope, 1), ToV8(&scope, "Hi, Mom!"))));
+ v8::Local<v8::Object> v8Object = v8::Object::New(scope.isolate());
+ EXPECT_TRUE(v8CallBoolean(v8Object->Set(
+ scope.context(), ToV8(&scope, "foo"), ToV8(&scope, v8StringArray))));
+
+ NonThrowableExceptionState exceptionState;
+ const auto& record =
+ NativeValueTraits<IDLRecord<IDLString, IDLSequence<IDLString>>>::
+ nativeValue(scope.isolate(), v8Object, exceptionState);
+ EXPECT_EQ(1U, record.size());
+ EXPECT_EQ("foo", record[0].first);
+ EXPECT_EQ("Hello, World!", record[0].second[0]);
+ EXPECT_EQ("Hi, Mom!", record[0].second[1]);
+ }
+}
+
+} // namespace
+
+} // namespace blink

Powered by Google App Engine
This is Rietveld 408576698