Refactor fxjs_v8 and add embeddertests for it.

fpdfsdk/src/jsapi/fxjs_v8_embeddertest.cpp

+// Copyright 2015 PDFium 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 "../../../core/include/fpdfapi/fpdf_parser.h"
+#include "../../../testing/embedder_test.h"
+#include "../../include/fsdk_mgr.h"
+#include "../../include/javascript/JS_Runtime.h"
+#include "../../include/jsapi/fxjs_v8.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace {
+
+const wchar_t kScript[] = L"fred = 7";
+
+}  // namespace
+
+class FXJSV8Embeddertest : public EmbedderTest {
+ public:
+  FXJSV8Embeddertest()
+      : m_pContext(nullptr), m_pRuntime(nullptr), m_pIsolate(nullptr) {}
+  ~FXJSV8Embeddertest() override {}
+
+  void SetUp() override {
+    EmbedderTest::SetUp();
+    m_pAllocator.reset(new JS_ArrayBufferAllocator());
+
+    v8::Isolate::CreateParams params;
+    params.array_buffer_allocator = m_pAllocator.get();
+    m_pIsolate = v8::Isolate::New(params);
+
+    v8::Isolate::Scope isolate_scope(m_pIsolate);
+    v8::HandleScope handle_scope(m_pIsolate);
+    JS_Initial(0);
+    JS_PrepareIsolate(m_pIsolate);

[Tom Sepez, 2015/09/14 23:42:52]

note: this is the important step, since PrepareIsolate expects that the isolate
has had a data structure grafted onto it.

+    JS_InitialRuntime(m_pIsolate, m_pRuntime, m_pContext, m_pPersistentContext);
+  }
+
+  void TearDown() override {
+    JS_ReleaseRuntime(m_pIsolate, m_pPersistentContext);
+    JS_ResetIsolate(m_pIsolate);
+    JS_Release();
+    EmbedderTest::TearDown();
+  }
+
+  IFXJS_Context* context() const { return m_pContext; }
+  IFXJS_Runtime* runtime() const { return m_pRuntime; }
+  v8::Isolate* isolate() const { return m_pIsolate; }
+  v8::Local<v8::Context> GetV8Context() {
+    return v8::Local<v8::Context>::New(m_pIsolate, m_pPersistentContext);
+  }
+
+ private:
+  IFXJS_Context* m_pContext;

[Lei Zhang, 2015/09/15 01:59:55]

(Just staring the at the code) Does the runtime/context ever get set to
non-NULL?

[Tom Sepez, 2015/09/15 15:56:42]

Not yet.  In the future, mock objects will need to get defined if we want to
test all the JS_Define stuff, and we'll need them then.  But that could be a
subclass of this test.  Removed.

+  IFXJS_Runtime* m_pRuntime;
+  v8::Isolate* m_pIsolate;
+  v8::Global<v8::Context> m_pPersistentContext;
+  nonstd::unique_ptr<v8::ArrayBuffer::Allocator> m_pAllocator;
+};
+
+TEST_F(FXJSV8Embeddertest, Getters) {
+  v8::Isolate::Scope isolate_scope(isolate());
+  v8::HandleScope handle_scope(isolate());
+  v8::Context::Scope context_scope(GetV8Context());
+
+  FXJSErr error;
+  CFX_WideString wsInfo;
+  CFX_WideString wsScript(kScript);
+  int sts = JS_Execute(isolate(), context(), kScript, wcslen(kScript), &error);
+  EXPECT_EQ(0, sts);
+
+  v8::Local<v8::Object> This = JS_GetThisObj(isolate());
+  v8::Local<v8::Value> fred = JS_GetObjectElement(isolate(), This, L"fred");
+  EXPECT_TRUE(fred->IsNumber());
+}