Issue 3593 - stop sharing v8::Isolate among tests

src/JsEngine.cpp

Index: src/JsEngine.cpp
diff --git a/src/JsEngine.cpp b/src/JsEngine.cpp
index d9b77228053edd5c27d6f753cd0aa534c71e984a..8fe35928c51d30faa7798a644a2ee19b5381d151 100644
--- a/src/JsEngine.cpp
+++ b/src/JsEngine.cpp
@@ -125,10 +125,8 @@ void JsEngine::CallTimerTask(const JsWeakValuesID& timerParamsID)
   callback.Call(timerParams);
 }
 
-AdblockPlus::JsEngine::JsEngine(const ScopedV8IsolatePtr& isolate,
-  TimerPtr timer, WebRequestPtr webRequest)
-  : isolate(isolate)
-  , fileSystem(new DefaultFileSystem())
+AdblockPlus::JsEngine::JsEngine(TimerPtr timer, WebRequestPtr webRequest)
+  : fileSystem(new DefaultFileSystem())
   , logSystem(new DefaultLogSystem())
   , timer(std::move(timer))
   , webRequest(std::move(webRequest))
@@ -136,10 +134,9 @@ AdblockPlus::JsEngine::JsEngine(const ScopedV8IsolatePtr& isolate,
 }
 
 AdblockPlus::JsEnginePtr AdblockPlus::JsEngine::New(const AppInfo& appInfo,
-  TimerPtr timer, WebRequestPtr webRequest,
-  const ScopedV8IsolatePtr& isolate)
+  TimerPtr timer, WebRequestPtr webRequest)
 {
-  JsEnginePtr result(new JsEngine(isolate, std::move(timer), std::move(webRequest)));
+  JsEnginePtr result(new JsEngine(std::move(timer), std::move(webRequest)));
 
   const v8::Locker locker(result->GetIsolate());
   const v8::Isolate::Scope isolateScope(result->GetIsolate());