Issue #3593 - Make isolate a fully internal member of the engine

src/JsEngineInternal.h

 
 #if !defined(ADBLOCK_PLUS_JS_ENGINE_INTERNAL_H)
 #define ADBLOCK_PLUS_JS_ENGINE_INTERNAL_H
 
 #include <AdblockPlus/JsEngine.h>
 #include <v8.h>
 #include <array>
 #include "AllocatedArray.h"
 #include "Scheduler.h"
 #include "V8Upgrade.h"
@@ skipping 76 matching lines @@
    *   to allocate a shared pointer to the task.
    *
    * @param task
    *   Task to execute. All v8 handles must refer to the present engine.
    * @param ImmediateSingleUseThreadType
    *   The schedule policy--create a new thread and discard it afterwards.
    */
   void ScheduleTaskInternal(std::shared_ptr<TaskFunctionInterface>&& body);
 
 public:
-  JsEngineInternal(const AdblockPlus::ScopedV8IsolatePtr& isolate);
+  JsEngineInternal(v8::Isolate* isolate);
 
   /**
    * Retrieve our persistent v8 context as a local handle.
    */
   v8::Local<v8::Context> GetContextAsLocal() const;
 
   /**
    * Retrieve the global object of our context.
    */
   v8::Local<v8::Object> GetGlobalObject();
@@ skipping 126 matching lines @@
    */
   const v8::HandleScope handleScope;
   const v8::Context::Scope contextScope;
 public:
   V8ExecutionScope(JsEngineInternal* engine);
 };
 
 
 
 #endif