Get CJS_RuntimeFactory out of the CJS_GlobalData management business.
[pdfium.git] / fpdfsdk / src / javascript / JS_Runtime.cpp
index 4ef34a8..8ba7e81 100644 (file)
@@ -50,7 +50,6 @@ void CJS_RuntimeFactory::Release() {
     // to do.Should be implemented as atom manipulation.
     if (--m_nRef == 0) {
       JS_Release();
-      ReleaseGlobalData();
       m_bInit = FALSE;
     }
   }
@@ -60,25 +59,6 @@ void CJS_RuntimeFactory::DeleteJSRuntime(IFXJS_Runtime* pRuntime) {
   delete (CJS_Runtime*)pRuntime;
 }
 
-CJS_GlobalData* CJS_RuntimeFactory::NewGlobalData(CPDFDoc_Environment* pApp) {
-  if (m_pGlobalData) {
-    m_nGlobalDataCount++;
-    return m_pGlobalData;
-  }
-  m_nGlobalDataCount = 1;
-  m_pGlobalData = new CJS_GlobalData(pApp);
-  return m_pGlobalData;
-}
-
-void CJS_RuntimeFactory::ReleaseGlobalData() {
-  m_nGlobalDataCount--;
-
-  if (m_nGlobalDataCount <= 0) {
-    delete m_pGlobalData;
-    m_pGlobalData = NULL;
-  }
-}
-
 void* CJS_ArrayBufferAllocator::Allocate(size_t length) {
   return calloc(1, length);
 }
@@ -113,10 +93,10 @@ CJS_Runtime::CJS_Runtime(CPDFDoc_Environment* pApp)
     m_isolateManaged = true;
   }
 
-  InitJSObjects();
+  DefineJSObjects();
 
   CJS_Context* pContext = (CJS_Context*)NewContext();
-  JS_InitialRuntime(*this, this, pContext, m_context);
+  JS_InitialRuntime(GetIsolate(), this, pContext, m_context);
   ReleaseContext(pContext);
 }
 
@@ -125,9 +105,7 @@ CJS_Runtime::~CJS_Runtime() {
     delete m_ContextArray.GetAt(i);
 
   m_ContextArray.RemoveAll();
-
-  JS_ReleaseRuntime(*this, m_context);
-
+  JS_ReleaseRuntime(GetIsolate(), m_context);
   RemoveEventsInLoop(m_pFieldEventPath);
 
   m_pApp = NULL;
@@ -139,68 +117,51 @@ CJS_Runtime::~CJS_Runtime() {
     m_isolate->Dispose();
 }
 
-FX_BOOL CJS_Runtime::InitJSObjects() {
+void CJS_Runtime::DefineJSObjects() {
   v8::Isolate::Scope isolate_scope(GetIsolate());
   v8::HandleScope handle_scope(GetIsolate());
   v8::Local<v8::Context> context = v8::Context::New(GetIsolate());
   v8::Context::Scope context_scope(context);
-  // 0 - 8
-  if (CJS_Border::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-  if (CJS_Display::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-  if (CJS_Font::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-  if (CJS_Highlight::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-  if (CJS_Position::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-  if (CJS_ScaleHow::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-  if (CJS_ScaleWhen::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-  if (CJS_Style::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-  if (CJS_Zoomtype::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-
-  // 9 - 11
-  if (CJS_App::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-  if (CJS_Color::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-  if (CJS_Console::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-
-  // 12 - 14
-  if (CJS_Document::Init(*this, JS_DYNAMIC) < 0)
-    return FALSE;
-  if (CJS_Event::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-  if (CJS_Field::Init(*this, JS_DYNAMIC) < 0)
-    return FALSE;
-
-  // 15 - 17
-  if (CJS_Global::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-  if (CJS_Icon::Init(*this, JS_DYNAMIC) < 0)
-    return FALSE;
-  if (CJS_Util::Init(*this, JS_STATIC) < 0)
-    return FALSE;
-
-  if (CJS_PublicMethods::Init(*this) < 0)
-    return FALSE;
-  if (CJS_GlobalConsts::Init(*this) < 0)
-    return FALSE;
-  if (CJS_GlobalArrays::Init(*this) < 0)
-    return FALSE;
-
-  if (CJS_TimerObj::Init(*this, JS_DYNAMIC) < 0)
-    return FALSE;
-  if (CJS_PrintParamsObj::Init(*this, JS_DYNAMIC) < 0)
-    return FALSE;
 
-  return TRUE;
+  // The call order determines the "ObjDefID" assigned to each class.
+  // ObjDefIDs 0 - 2
+  CJS_Border::DefineJSObjects(GetIsolate(), JS_STATIC);
+  CJS_Display::DefineJSObjects(GetIsolate(), JS_STATIC);
+  CJS_Font::DefineJSObjects(GetIsolate(), JS_STATIC);
+
+  // ObjDefIDs 3 - 5
+  CJS_Highlight::DefineJSObjects(GetIsolate(), JS_STATIC);
+  CJS_Position::DefineJSObjects(GetIsolate(), JS_STATIC);
+  CJS_ScaleHow::DefineJSObjects(GetIsolate(), JS_STATIC);
+
+  // ObjDefIDs 6 - 8
+  CJS_ScaleWhen::DefineJSObjects(GetIsolate(), JS_STATIC);
+  CJS_Style::DefineJSObjects(GetIsolate(), JS_STATIC);
+  CJS_Zoomtype::DefineJSObjects(GetIsolate(), JS_STATIC);
+
+  // ObjDefIDs 9 - 11
+  CJS_App::DefineJSObjects(GetIsolate(), JS_STATIC);
+  CJS_Color::DefineJSObjects(GetIsolate(), JS_STATIC);
+  CJS_Console::DefineJSObjects(GetIsolate(), JS_STATIC);
+
+  // ObjDefIDs 12 - 14
+  CJS_Document::DefineJSObjects(GetIsolate(), JS_DYNAMIC);
+  CJS_Event::DefineJSObjects(GetIsolate(), JS_STATIC);
+  CJS_Field::DefineJSObjects(GetIsolate(), JS_DYNAMIC);
+
+  // ObjDefIDs 15 - 17
+  CJS_Global::DefineJSObjects(GetIsolate(), JS_STATIC);
+  CJS_Icon::DefineJSObjects(GetIsolate(), JS_DYNAMIC);
+  CJS_Util::DefineJSObjects(GetIsolate(), JS_STATIC);
+
+  // ObjDefIDs 18 - 20 (these can't fail, return void).
+  CJS_PublicMethods::DefineJSObjects(GetIsolate());
+  CJS_GlobalConsts::DefineJSObjects(GetIsolate());
+  CJS_GlobalArrays::DefineJSObjects(GetIsolate());
+
+  // ObjDefIDs 21 - 22.
+  CJS_TimerObj::DefineJSObjects(GetIsolate(), JS_DYNAMIC);
+  CJS_PrintParamsObj::DefineJSObjects(GetIsolate(), JS_DYNAMIC);
 }
 
 IFXJS_Context* CJS_Runtime::NewContext() {
@@ -236,20 +197,17 @@ void CJS_Runtime::SetReaderDocument(CPDFSDK_Document* pReaderDoc) {
     v8::Context::Scope context_scope(context);
 
     m_pDocument = pReaderDoc;
-
     if (pReaderDoc) {
-      JSObject pThis = JS_GetThisObj(*this);
+      v8::Local<v8::Object> pThis = JS_GetThisObj(GetIsolate());
       if (!pThis.IsEmpty()) {
-        if (JS_GetObjDefnID(pThis) == JS_GetObjDefnID(*this, L"Document")) {
+        if (JS_GetObjDefnID(pThis) ==
+            JS_GetObjDefnID(GetIsolate(), L"Document")) {
           if (CJS_Document* pJSDocument = (CJS_Document*)JS_GetPrivate(pThis)) {
             if (Document* pDocument = (Document*)pJSDocument->GetEmbedObject())
               pDocument->AttachDoc(pReaderDoc);
           }
         }
       }
-      JS_SetThisObj(*this, JS_GetObjDefnID(*this, L"Document"));
-    } else {
-      JS_SetThisObj(*this, JS_GetObjDefnID(*this, L"app"));
     }
   }
 }