From 9ad9a5fc81a36ab406ff49408dd0814d350bbc7e Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Wed, 7 Feb 2018 21:07:24 +0000 Subject: Split creation of ordinary object and bound objects in FXJS. One can be performed by the CJS_V8 layer, the other requires the full FXJS mechanism. Avoids using -1 as a special case. Change-Id: I4a14ccb6a7fea393f84b70a07ada03b1a83c7d36 Reviewed-on: https://pdfium-review.googlesource.com/25830 Reviewed-by: dsinclair Commit-Queue: Tom Sepez --- fxjs/fxjs_v8_embeddertest.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'fxjs/fxjs_v8_embeddertest.cpp') diff --git a/fxjs/fxjs_v8_embeddertest.cpp b/fxjs/fxjs_v8_embeddertest.cpp index 9e90663aa2..acc4366631 100644 --- a/fxjs/fxjs_v8_embeddertest.cpp +++ b/fxjs/fxjs_v8_embeddertest.cpp @@ -226,7 +226,7 @@ TEST_F(FXJSV8EmbedderTest, NewObject) { v8::HandleScope handle_scope(isolate()); v8::Context::Scope context_scope(GetV8Context()); - auto object = engine()->NewFxDynamicObj(-1); + auto object = engine()->NewObject(); ASSERT_FALSE(object.IsEmpty()); EXPECT_EQ(0u, engine()->GetObjectPropertyNames(object).size()); EXPECT_FALSE(engine()->GetObjectProperty(object, L"clams").IsEmpty()); -- cgit v1.2.3