diff options
| author | Tom Sepez <tsepez@chromium.org> | 2018-02-05 15:13:09 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-02-05 15:13:09 +0000 |
| commit | b5b7972cdc064498b45d230225aa92fee1acc7b6 (patch) | |
| tree | 544ffaf0db87f3538a0d3caee2640c55483f970d /fxjs/fxjs_v8.cpp | |
| parent | b1d7872faacbd37c03eacc1e86214904db8f658f (diff) | |
| download | pdfium-b5b7972cdc064498b45d230225aa92fee1acc7b6.tar.xz | |
Move const arrays to CFXJS_Engine from CJS_V8
CJS_V8 is shared by both the non-xfa and xfa-side JS engines,
but the const arrays are only used by the non-xfa side.
Change-Id: Ic5ed8238df1a14dde8a4463b2388b4a7a923e392
Reviewed-on: https://pdfium-review.googlesource.com/25250
Commit-Queue: dsinclair <dsinclair@chromium.org>
Reviewed-by: dsinclair <dsinclair@chromium.org>
Diffstat (limited to 'fxjs/fxjs_v8.cpp')
| -rw-r--r-- | fxjs/fxjs_v8.cpp | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/fxjs/fxjs_v8.cpp b/fxjs/fxjs_v8.cpp index c9d227cc47..f1555b2e96 100644 --- a/fxjs/fxjs_v8.cpp +++ b/fxjs/fxjs_v8.cpp @@ -436,7 +436,7 @@ void CFXJS_Engine::ReleaseEngine() { if (!pData) return; - ClearConstArray(); + m_ConstArrays.clear(); int maxID = CFXJS_ObjDefinition::MaxID(GetIsolate()); for (int i = 0; i < maxID; ++i) { @@ -561,3 +561,12 @@ void* CFXJS_Engine::GetObjectPrivate(v8::Local<v8::Object> pObj) { } return pData ? pData->m_pPrivate : nullptr; } + +v8::Local<v8::Array> CFXJS_Engine::GetConstArray(const WideString& name) { + return v8::Local<v8::Array>::New(GetIsolate(), m_ConstArrays[name]); +} + +void CFXJS_Engine::SetConstArray(const WideString& name, + v8::Local<v8::Array> array) { + m_ConstArrays[name] = v8::Global<v8::Array>(GetIsolate(), array); +} |