From 91cd4febe4c959d8256341e25a59a4d9dca9ca69 Mon Sep 17 00:00:00 2001 From: "jochen@chromium.org" Date: Thu, 28 Nov 2013 09:31:58 +0000 Subject: Remove usage of deprecagted V8 API from gin/ R=mkwst@chromium.org TBR=abarth@chromium.org BUG=324225 Review URL: https://codereview.chromium.org/94413002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@237750 0039d316-1c4b-4281-b951-d872f2087c98 --- gin/converter_unittest.cc | 56 +++++++++++++++++++++++------------------------ 1 file changed, 28 insertions(+), 28 deletions(-) (limited to 'gin/converter_unittest.cc') diff --git a/gin/converter_unittest.cc b/gin/converter_unittest.cc index 58ae625..9b831b9 100644 --- a/gin/converter_unittest.cc +++ b/gin/converter_unittest.cc @@ -34,25 +34,25 @@ TEST_F(ConverterTest, Bool) { HandleScope handle_scope(instance_->isolate()); EXPECT_TRUE(Converter::ToV8(instance_->isolate(), true)->StrictEquals( - Boolean::New(true))); + Boolean::New(instance_->isolate(), true))); EXPECT_TRUE(Converter::ToV8(instance_->isolate(), false)->StrictEquals( - Boolean::New(false))); + Boolean::New(instance_->isolate(), false))); struct { Handle input; bool expected; } test_data[] = { - { Boolean::New(false).As(), false }, - { Boolean::New(true).As(), true }, - { Number::New(0).As(), false }, - { Number::New(1).As(), true }, - { Number::New(-1).As(), true }, - { Number::New(0.1).As(), true }, - { String::New("").As(), false }, - { String::New("foo").As(), true }, - { Object::New().As(), true }, - { Null().As(), false }, - { Undefined().As(), false }, + { Boolean::New(instance_->isolate(), false).As(), false }, + { Boolean::New(instance_->isolate(), true).As(), true }, + { Number::New(instance_->isolate(), 0).As(), false }, + { Number::New(instance_->isolate(), 1).As(), true }, + { Number::New(instance_->isolate(), -1).As(), true }, + { Number::New(instance_->isolate(), 0.1).As(), true }, + { String::NewFromUtf8(instance_->isolate(), "").As(), false }, + { String::NewFromUtf8(instance_->isolate(), "foo").As(), true }, + { Object::New(instance_->isolate()).As(), true }, + { Null(instance_->isolate()).As(), false }, + { Undefined(instance_->isolate()).As(), false }, }; for (size_t i = 0; i < ARRAYSIZE_UNSAFE(test_data); ++i) { @@ -82,19 +82,19 @@ TEST_F(ConverterTest, Int32) { bool expect_sucess; int expected_result; } test_data_from[] = { - { Boolean::New(false).As(), false, 0 }, - { Boolean::New(true).As(), false, 0 }, - { Integer::New(-1).As(), true, -1 }, - { Integer::New(0).As(), true, 0 }, - { Integer::New(1).As(), true, 1 }, - { Number::New(-1).As(), true, -1 }, - { Number::New(1.1).As(), false, 0 }, - { String::New("42").As(), false, 0 }, - { String::New("foo").As(), false, 0 }, - { Object::New().As(), false, 0 }, - { Array::New().As(), false, 0 }, - { v8::Null().As(), false, 0 }, - { v8::Undefined().As(), false, 0 }, + { Boolean::New(instance_->isolate(), false).As(), false, 0 }, + { Boolean::New(instance_->isolate(), true).As(), false, 0 }, + { Integer::New(instance_->isolate(), -1).As(), true, -1 }, + { Integer::New(instance_->isolate(), 0).As(), true, 0 }, + { Integer::New(instance_->isolate(), 1).As(), true, 1 }, + { Number::New(instance_->isolate(), -1).As(), true, -1 }, + { Number::New(instance_->isolate(), 1.1).As(), false, 0 }, + { String::NewFromUtf8(instance_->isolate(), "42").As(), false, 0 }, + { String::NewFromUtf8(instance_->isolate(), "foo").As(), false, 0 }, + { Object::New(instance_->isolate()).As(), false, 0 }, + { Array::New(instance_->isolate()).As(), false, 0 }, + { v8::Null(instance_->isolate()).As(), false, 0 }, + { v8::Undefined(instance_->isolate()).As(), false, 0 }, }; for (size_t i = 0; i < ARRAYSIZE_UNSAFE(test_data_from); ++i) { @@ -120,8 +120,8 @@ TEST_F(ConverterTest, Vector) { ASSERT_FALSE(js_array.IsEmpty()); EXPECT_EQ(3u, js_array->Length()); for (size_t i = 0; i < expected.size(); ++i) { - EXPECT_TRUE(Integer::New(expected[i])->StrictEquals( - js_array->Get(static_cast(i)))); + EXPECT_TRUE(Integer::New(instance_->isolate(), expected[i]) + ->StrictEquals(js_array->Get(static_cast(i)))); } std::vector actual; -- cgit v1.1