Pass IJS_Runtime, not IJS_Context, to native object constructors.

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 83e2c9af2a4984c8c6dafab01b6fe7b912a94684..3215c399e8130c473989627f57a92cef48e71c53 100644
--- a/fpdfsdk/src/javascript/JS_Runtime.cpp
+++ b/fpdfsdk/src/javascript/JS_Runtime.cpp
@@ -61,7 +61,7 @@ CJS_Runtime::CJS_Runtime(CPDFDoc_Environment* pApp)
     DefineJSObjects();
 
   CJS_Context* pContext = (CJS_Context*)NewContext();
-  FXJS_InitializeRuntime(GetIsolate(), this, pContext, m_context);
+  FXJS_InitializeRuntime(GetIsolate(), this, m_context);
   ReleaseContext(pContext);
 }
 
@@ -178,6 +178,19 @@ void CJS_Runtime::SetReaderDocument(CPDFSDK_Document* pReaderDoc) {
   }
 }
 
+int CJS_Runtime::Execute(IJS_Context* cc,
+                         const wchar_t* script,
+                         long length,
+                         CFX_WideString& info) {
+  FXJSErr error = {};
+  int nRet = FXJS_Execute(m_isolate, cc, script, length, &error);

[Lei Zhang, 2015/10/08 23:52:01]

FXJS_Execute() doesn't even use the length argument. Drop it?

[Tom Sepez, 2015/10/09 00:14:20]

Done.

+  if (nRet < 0) {
+    info.Format(L"[ Line: %05d { %s } ] : %s", error.linnum - 1, error.srcline,
+                error.message);
+  }
+  return nRet;
+}
+
 bool CJS_Runtime::AddEventToSet(const FieldEvent& event) {
   return m_FieldEventSet.insert(event).second;
 }