diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-02-02 17:37:37 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-02-02 17:37:37 +0000 |
| commit | 1258f7f21725c703269581e0241fbad86d89209c (patch) | |
| tree | 4b065d6c432ee9e23995cab693a9eab73fbddfa1 /fxjs/cjs_event_context.cpp | |
| parent | 9cbd2dd8ff0812aae57a99d6a7dc285cc8b9e262 (diff) | |
| download | pdfium-1258f7f21725c703269581e0241fbad86d89209c.tar.xz | |
Remove V8 context from CJS_V8
Makes CFXJS_Engine and CFXJSE_Engine consistent with each other in
that both have a V8 context to themselves, and not one inheritted
from the CJS_V8 (which is now more per-isolate than per-context).
Consolidate NewLocalContext() and GetPersistentContext(), which both
did the exact same thing under the covers once inside v8 land.
Rename a few things to make it simpler.
Change-Id: I68905db9ad44253063da235fcb276a75627a2dbc
Reviewed-on: https://pdfium-review.googlesource.com/25170
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Diffstat (limited to 'fxjs/cjs_event_context.cpp')
| -rw-r--r-- | fxjs/cjs_event_context.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fxjs/cjs_event_context.cpp b/fxjs/cjs_event_context.cpp index d2f270b1c4..195fb1b888 100644 --- a/fxjs/cjs_event_context.cpp +++ b/fxjs/cjs_event_context.cpp @@ -28,7 +28,7 @@ CPDFSDK_FormFillEnvironment* CJS_EventContext::GetFormFillEnv() { bool CJS_EventContext::RunScript(const WideString& script, WideString* info) { v8::Isolate::Scope isolate_scope(m_pRuntime->GetIsolate()); v8::HandleScope handle_scope(m_pRuntime->GetIsolate()); - v8::Local<v8::Context> context = m_pRuntime->NewLocalContext(); + v8::Local<v8::Context> context = m_pRuntime->GetV8Context(); v8::Context::Scope context_scope(context); if (m_bBusy) { |