Plumb in an externally created v8::Isolate

fpdfsdk/src/javascript/JS_Runtime.cpp

Index: fpdfsdk/src/javascript/JS_Runtime.cpp
diff --git a/fpdfsdk/src/javascript/JS_Runtime.cpp b/fpdfsdk/src/javascript/JS_Runtime.cpp
index c2c1a5140070ec387e6a6be207cd4d5e09306ca0..ae4b3357b657f46b2a117a4584d83350295d051a 100644
--- a/fpdfsdk/src/javascript/JS_Runtime.cpp
+++ b/fpdfsdk/src/javascript/JS_Runtime.cpp
@@ -112,11 +112,15 @@ CJS_Runtime::CJS_Runtime(CPDFDoc_Environment * pApp) :
 	m_bRegistered(FALSE),
 	m_pFieldEventPath(NULL)
 {
-	m_pArrayBufferAllocator.reset(new CJS_ArrayBufferAllocator());
-
-	v8::Isolate::CreateParams params;
-	params.array_buffer_allocator = m_pArrayBufferAllocator.get();
-	m_isolate = v8::Isolate::New(params);
+	if (m_pApp->GetFormFillInfo()->m_pJsPlatform->version == 2) {

[Tom Sepez, 2015/07/14 16:22:53]

I think we want >= 2 for future compatibility. Also, how about allowing callers
to pass a null m_isolate to get the current behaviour? 

+		m_isolate = m_pApp->GetFormFillInfo()->m_pJsPlatform->m_isolate;
+	} else {
+		m_pArrayBufferAllocator.reset(new CJS_ArrayBufferAllocator());
+
+		v8::Isolate::CreateParams params;
+		params.array_buffer_allocator = m_pArrayBufferAllocator.get();
+		m_isolate = v8::Isolate::New(params);
+        }
 
 	InitJSObjects();