XFA: contention between FXJSE and FXJS over isolate data slots

fpdfsdk/src/jsapi/fxjs_v8.cpp

Index: fpdfsdk/src/jsapi/fxjs_v8.cpp
diff --git a/fpdfsdk/src/jsapi/fxjs_v8.cpp b/fpdfsdk/src/jsapi/fxjs_v8.cpp
index 135b5e93498c19c5e839c76f14a73f0d2049e96e..4634f1baa951b178e8052e950a2efb90ec69e0c7 100644
--- a/fpdfsdk/src/jsapi/fxjs_v8.cpp
+++ b/fpdfsdk/src/jsapi/fxjs_v8.cpp
@@ -273,8 +273,6 @@ void JS_InitializeRuntime(v8::Isolate* pIsolate,
       v8::Local<v8::ObjectTemplate>::New(pIsolate, globalObjTemp));
   v8::Context::Scope context_scope(v8Context);
 
-  // v8::Local<External> ptr = External::New(isolate, pFXRuntime);
-  // v8Context->SetEmbedderData(1, ptr);
   // TODO(tsepez): Don't use more than one embedder data slot.
   pIsolate->SetData(2, pFXRuntime);
 
@@ -349,7 +347,6 @@ void JS_ReleaseRuntime(v8::Isolate* pIsolate,
   }
   delete pArray;
 
-  pIsolate->SetData(1, NULL);
   pIsolate->SetData(g_embedderDataSlot, NULL);
   // TODO(tsepez): Don't use more than one embedder data slot.
   pIsolate->SetData(2, NULL);