diff options
| author | weili <weili@chromium.org> | 2016-09-21 11:50:43 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-09-21 11:50:43 -0700 |
| commit | 0b2a9874bddf44b4226fcbafa9ce159a9b3735a8 (patch) | |
| tree | ddd52af69935f9c5480386dfbfbc923c03b6e71e /fxjs/fxjs_v8_embeddertest.cpp | |
| parent | 54be7be992f1cde40b9d5c0e55a119c6cc4e4e02 (diff) | |
| download | pdfium-0b2a9874bddf44b4226fcbafa9ce159a9b3735a8.tar.xz | |
Set up isolate in CFXJS_Engine's constructor
CFXJS_Engine class should always be constructed with an isolate, except
for its subclasses which may need to create an isolate by themselves.
Move SetIsolate() function to be protected so that only subclasses can
access it.
Review-Url: https://codereview.chromium.org/2354353002
Diffstat (limited to 'fxjs/fxjs_v8_embeddertest.cpp')
| -rw-r--r-- | fxjs/fxjs_v8_embeddertest.cpp | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/fxjs/fxjs_v8_embeddertest.cpp b/fxjs/fxjs_v8_embeddertest.cpp index 2da313192d..a877a36e56 100644 --- a/fxjs/fxjs_v8_embeddertest.cpp +++ b/fxjs/fxjs_v8_embeddertest.cpp @@ -45,12 +45,10 @@ TEST_F(FXJSV8EmbedderTest, MultipleEngines) { v8::Isolate::Scope isolate_scope(isolate()); v8::HandleScope handle_scope(isolate()); - CFXJS_Engine engine1; - engine1.SetIsolate(isolate()); + CFXJS_Engine engine1(isolate()); engine1.InitializeEngine(); - CFXJS_Engine engine2; - engine2.SetIsolate(isolate()); + CFXJS_Engine engine2(isolate()); engine2.InitializeEngine(); v8::Context::Scope context_scope(GetV8Context()); |