Issue #4692 - Rewrite web request task to avoid engine self-reference

src/JsEngine.cpp

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 #include <AdblockPlus.h>
 #include "GlobalJsObject.h"
 #include "JsContext.h"
 #include "JsEngineInternal.h"
 #include "JsEngineTransition.h"
 #include "JsError.h"
 #include "Scheduler.h"
 #include "Utils.h"
+#include "WebRequestJsObject.h"
 
 const AdblockPlus::ImmediateSingleUseThreadType AdblockPlus::ImmediateSingleUseThread = {};
 
 class AdblockPlus::JsEngine::SchedulerImpl
   : public SchedulerT<SingleUseWorker>
 {
 public:
   SchedulerImpl() {} // = default;
 };
 
@@ skipping 66 matching lines @@
 
 JsEngineInternal::JsEngineInternal(const AdblockPlus::ScopedV8IsolatePtr& isolate)
   : AdblockPlus::JsEngine(isolate),
   context(isolate->Get(),v8::Context::New(isolate->Get()))
 {
   /*
    * Enter v8 scope for our context so that we can initialize its global object.
    */
   const v8::Context::Scope contextScope(GetContextAsLocal());
   auto globalObject = GetGlobalObject();
+  // Timeout
   auto propertyName = AdblockPlus::Utils::ToV8String(GetIsolate(), "setTimeout");
   globalObject->Set(propertyName, MakeCallback(::CallbackForSetTimeout));
+  // Web request
+  auto auxiliaryObject = v8::Object::New();
+  propertyName = AdblockPlus::Utils::ToV8String(GetIsolate(), "GET");
+  auxiliaryObject->Set(propertyName, MakeCallback(::GETCallback));
+  propertyName = AdblockPlus::Utils::ToV8String(GetIsolate(), "_webRequest");
+  globalObject->Set(propertyName, auxiliaryObject);
   // TODO: Move the rest of the global object initializations here
 }
 
 /**
  * \par Design Notes
  * It is technically necessary to construct JsEngine instances *only* within a factory.
  * Initialization requires that certain transient v8 scopes be set up 
  *   before initialization and torn down afterwards.
  * C++ has no syntax to use anything like a sentry object in the constructor itself.
  * Thus we need to establish v8 scope within every C++ scope that constructs an object.
@@ skipping 252 matching lines @@
 
 JsEngineInternal* ToInternal(AdblockPlus::JsEnginePtr p)
 {
   return static_cast<JsEngineInternal*>(p.get());
 }
 
 JsEngineInternal* ToInternal(AdblockPlus::JsEngine* p)
 {
   return static_cast<JsEngineInternal*>(p);
 }