Bug 1491558 follow-up: Address mats' review comments

dom/crypto/WebCryptoThreadPool.cpp

@@ -74,7 +74,6 @@ WebCryptoThreadPool::DispatchInternal(nsIRunnable* aRunnable)
     NS_ENSURE_TRUE(EnsureNSSInitializedChromeOrContent(), NS_ERROR_FAILURE);
 
     nsCOMPtr<nsIThreadPool> pool(new nsThreadPool());
-    NS_ENSURE_TRUE(pool, NS_ERROR_FAILURE);
 
     nsresult rv = pool->SetName(NS_LITERAL_CSTRING("SubtleCrypto"));
     NS_ENSURE_SUCCESS(rv, rv);

netwerk/base/nsStreamTransportService.cpp

@@ -205,7 +205,6 @@ nsresult
 nsStreamTransportService::Init()
 {
     mPool = new nsThreadPool();
-    NS_ENSURE_STATE(mPool);
 
     // Configure the pool
     mPool->SetName(NS_LITERAL_CSTRING("StreamTrans"));

xpcom/tests/gtest/TestEventTargetQI.cpp

@@ -21,7 +21,6 @@ using namespace mozilla;
 TEST(TestEventTargetQI, ThreadPool)
 {
   nsCOMPtr<nsIThreadPool> thing = new nsThreadPool();
-  EXPECT_TRUE(thing);
 
   nsCOMPtr<nsISerialEventTarget> serial = do_QueryInterface(thing);
   EXPECT_FALSE(serial);

xpcom/tests/gtest/TestThreadPool.cpp

@@ -50,7 +50,6 @@ mozilla::Atomic<int> Task::sCount;
 TEST(ThreadPool, Main)
 {
   nsCOMPtr<nsIThreadPool> pool = new nsThreadPool();
-  EXPECT_TRUE(pool);
 
   for (int i = 0; i < 100; ++i) {
     nsCOMPtr<nsIRunnable> task = new Task(i);
@@ -66,7 +65,6 @@ TEST(ThreadPool, Main)
 TEST(ThreadPool, Parallelism)
 {
   nsCOMPtr<nsIThreadPool> pool = new nsThreadPool();
-  EXPECT_TRUE(pool);
 
   // Dispatch and sleep to ensure we have an idle thread
   nsCOMPtr<nsIRunnable> r0 = new Runnable("TestRunnable");