Issue 4832 - remove API allowing changing of LogSystem after Initialization of JsEngine

test/BaseJsTest.cpp

 /*
 * This file is part of Adblock Plus <https://adblockplus.org/>,
 * Copyright (C) 2006-2017 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
@@ skipping 15 matching lines @@
     {
       ii->callback();
       ii = timerTasks->erase(ii);
     }
     else
       ++ii;
   }
 }
 
 JsEngineCreationParameters::JsEngineCreationParameters()
-  : logSystem(std::make_shared<ThrowingLogSystem>())
+  : logSystem(new ThrowingLogSystem())
   , timer(new ThrowingTimer())
   , webRequest(new ThrowingWebRequest())
   , fileSystem(std::make_shared<ThrowingFileSystem>())
 {
 }
 
 AdblockPlus::JsEnginePtr CreateJsEngine(JsEngineCreationParameters&& jsEngineCreationParameters)
 {
   auto jsEngine = AdblockPlus::JsEngine::New(jsEngineCreationParameters.appInfo,
     std::move(jsEngineCreationParameters.timer),
     std::move(jsEngineCreationParameters.fileSystem),
-    std::move(jsEngineCreationParameters.webRequest));
-  jsEngine->SetLogSystem(std::move(jsEngineCreationParameters.logSystem));
+    std::move(jsEngineCreationParameters.webRequest),
+    std::move(jsEngineCreationParameters.logSystem));
   return jsEngine;
 }