Noissue - Fix sign comparison warnings in tests.

test/WebRequest.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
@@ skipping 106 matching lines @@
 {
   std::atomic<int> isConnectionAllowedCalledTimes(0);
   jsEngine->SetIsConnectionAllowedCallback([&isConnectionAllowedCalledTimes]()->bool
   {
     ++isConnectionAllowedCalledTimes;
     return true;
   });
   jsEngine->Evaluate("_webRequest.GET('http://example.com/', {X: 'Y'}, function(result) {foo = result;} )");
   ASSERT_TRUE(jsEngine->Evaluate("this.foo")->IsUndefined());
   AdblockPlus::Sleep(200);
-  EXPECT_EQ(1u, isConnectionAllowedCalledTimes);
+  EXPECT_EQ(1, isConnectionAllowedCalledTimes);
   EXPECT_EQ(AdblockPlus::WebRequest::NS_OK, jsEngine->Evaluate("foo.status")->AsInt());
   EXPECT_EQ(123, jsEngine->Evaluate("foo.responseStatus")->AsInt());
   EXPECT_EQ("http://example.com/\nX\nY", jsEngine->Evaluate("foo.responseText")->AsString());
   EXPECT_EQ("{\"Foo\":\"Bar\"}", jsEngine->Evaluate("JSON.stringify(foo.responseHeaders)")->AsString());
 }
 
 TEST_F(MockWebRequestTest, ConnectionIsNotAllowedOnJsEngine)
 {
   std::atomic<int> isConnectionAllowedCalledTimes(0);
   jsEngine->SetIsConnectionAllowedCallback([&isConnectionAllowedCalledTimes]()->bool
   {
     ++isConnectionAllowedCalledTimes;
     return false;
   });
   jsEngine->Evaluate("_webRequest.GET('http://example.com/', {X: 'Y'}, function(result) {foo = result;} )");
   ASSERT_TRUE(jsEngine->Evaluate("this.foo")->IsUndefined());
   AdblockPlus::Sleep(200);
-  EXPECT_EQ(1u, isConnectionAllowedCalledTimes);
+  EXPECT_EQ(1, isConnectionAllowedCalledTimes);
   EXPECT_EQ(AdblockPlus::WebRequest::NS_ERROR_CONNECTION_REFUSED, jsEngine->Evaluate("foo.status")->AsInt());
   EXPECT_EQ(0, jsEngine->Evaluate("foo.responseStatus")->AsInt());
   EXPECT_EQ("", jsEngine->Evaluate("foo.responseText")->AsString());
   EXPECT_EQ("{}", jsEngine->Evaluate("JSON.stringify(foo.responseHeaders)")->AsString());
 }
 
 namespace
 {
   class SyncStrings
   {
@@ skipping 321 matching lines @@
   catchLogSystem->clear();
   ResetTestXHR(jsEngine);
   jsEngine->Evaluate("\
     request.setRequestHeader('Security', 'theater');\nrequest.send();");
   EXPECT_EQ(AdblockPlus::LogSystem::LOG_LEVEL_TRACE, catchLogSystem->lastLogLevel);
   EXPECT_EQ("", catchLogSystem->lastMessage);
   WaitForVariable("result", jsEngine);
   EXPECT_FALSE(webRequest->lastRequestHeaders.cend() ==
                webRequest->lastRequestHeaders.find("Security"));
 }