Issue 5552 - use strict mode for all JS in libadblocklus

test/JsEngine.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 52 matching lines @@
 }
 
 namespace {
 
   bool IsSame(AdblockPlus::JsEngine& jsEngine,
               const AdblockPlus::JsValue& v1, const AdblockPlus::JsValue& v2)
   {
     AdblockPlus::JsValueList params;
     params.push_back(v1);
     params.push_back(v2);
-    return jsEngine.Evaluate("f = function(a, b) { return a == b };").Call(params).AsBool();
+    return jsEngine.Evaluate("(function(a, b) { return a == b })").Call(params).AsBool();
   }
 
 }
 
 TEST_F(JsEngineTest, ValueCopy)
 {
   {
     auto value = GetJsEngine().NewValue("foo");
     ASSERT_TRUE(value.IsString());
     ASSERT_EQ("foo", value.AsString());
@@ skipping 107 matching lines @@
   // v8::Isolate by default requires 32MB (depends on platform), so if there is
   // a memory leak than we will run out of memory on 32 bit platform because it
   // will allocate 32000 MB which is less than 2GB where it reaches out of
   // memory. Even on android where it allocates initially 16MB, the test still
   // makes sense.
   for (int i = 0; i < 1000; ++i)
   {
     JsEngine::New(AppInfo(), platform);
   }
 }