Bug 1203357 - Fix MSVC warning in dom/workers/test/gtest/TestReadWrite.cpp. r=baku.

The particular warning here was complaining about an unsafe comparison between
a uint32_t and a bool.

Also, remove ALLOW_COMPILER_WARNINGS=True for this directory.

--HG--
extra : rebase_source : e2ce5c425a48bf9cf29182f222d1f2dd44eb9bc8

dom/workers/test/gtest/TestReadWrite.cpp

@@ -248,7 +248,7 @@ TEST(ServiceWorkerRegistrar, TestWriteData)
     const mozilla::ipc::ContentPrincipalInfo& cInfo = data[i].principal();
 
     ASSERT_EQ((uint32_t)i, cInfo.appId());
-    ASSERT_EQ((uint32_t)(i %2), cInfo.isInBrowserElement());
+    ASSERT_EQ((uint32_t)(i % 2), (uint32_t)cInfo.isInBrowserElement());
 
     test.AppendPrintf("spec write %d", i);
     ASSERT_STREQ(test.get(), cInfo.spec().get());

dom/workers/test/gtest/moz.build

@@ -10,7 +10,4 @@ UNIFIED_SOURCES = [
 
 include('/ipc/chromium/chromium-config.mozbuild')
 
-# XXX: We should fix these warnings.
-ALLOW_COMPILER_WARNINGS = True
-
 FINAL_LIBRARY = 'xul-gtest'