Don't dispose the isolate if its provided by the embedder
authorJochen Eisinger <jochen@chromium.org>
Wed, 5 Aug 2015 06:55:39 +0000 (08:55 +0200)
committerJochen Eisinger <jochen@chromium.org>
Wed, 5 Aug 2015 06:55:39 +0000 (08:55 +0200)
BUG=chromium:515718
R=thestig@chromium.org

Review URL: https://codereview.chromium.org/1269783005 .

fpdfsdk/include/javascript/JS_Runtime.h
fpdfsdk/src/javascript/JS_Runtime.cpp

index f4dabf0..3be0f76 100644 (file)
@@ -68,6 +68,7 @@ class CJS_Runtime : public IFXJS_Runtime {
   CJS_FieldEvent* m_pFieldEventPath;
 
   v8::Isolate* m_isolate;
+  bool m_isolateManaged;
   nonstd::unique_ptr<CJS_ArrayBufferAllocator> m_pArrayBufferAllocator;
   v8::Global<v8::Context> m_context;
 };
index 08cd87c..4ef34a8 100644 (file)
@@ -98,7 +98,8 @@ CJS_Runtime::CJS_Runtime(CPDFDoc_Environment* pApp)
       m_pDocument(NULL),
       m_bBlocking(FALSE),
       m_pFieldEventPath(NULL),
-      m_isolate(NULL) {
+      m_isolate(NULL),
+      m_isolateManaged(false) {
   if (m_pApp->GetFormFillInfo()->m_pJsPlatform->version >= 2) {
     m_isolate = reinterpret_cast<v8::Isolate*>(
         m_pApp->GetFormFillInfo()->m_pJsPlatform->m_isolate);
@@ -109,6 +110,7 @@ CJS_Runtime::CJS_Runtime(CPDFDoc_Environment* pApp)
     v8::Isolate::CreateParams params;
     params.array_buffer_allocator = m_pArrayBufferAllocator.get();
     m_isolate = v8::Isolate::New(params);
+    m_isolateManaged = true;
   }
 
   InitJSObjects();
@@ -133,8 +135,8 @@ CJS_Runtime::~CJS_Runtime() {
   m_pFieldEventPath = NULL;
   m_context.Reset();
 
-  // m_isolate->Exit();
-  m_isolate->Dispose();
+  if (m_isolateManaged)
+    m_isolate->Dispose();
 }
 
 FX_BOOL CJS_Runtime::InitJSObjects() {