Replace EXPECT_EQ(NULL, foo) with EXPECT_TRUE(NULL == foo)....

net/socket/client_socket_pool_base_unittest.cc

-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
+// Copyright (c) 2009 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/socket/client_socket_pool_base.h"
 
 #include "base/compiler_specific.h"
 #include "base/message_loop.h"
 #include "base/platform_thread.h"
 #include "base/scoped_vector.h"
 #include "net/base/load_log.h"
@@ skipping 321 matching lines @@
 
 class TestConnectJobDelegate : public ConnectJob::Delegate {
  public:
   TestConnectJobDelegate()
       : have_result_(false), waiting_for_result_(false), result_(OK) {}
   virtual ~TestConnectJobDelegate() {}
 
   virtual void OnConnectJobComplete(int result, ConnectJob* job) {
     result_ = result;
     scoped_ptr<ClientSocket> socket(job->ReleaseSocket());
-    if (result == OK) {
-      EXPECT_TRUE(socket.get() != NULL);
-    } else {
-      EXPECT_EQ(NULL, socket.get());
-    }
+    // socket.get() should be NULL iff result != OK
+    EXPECT_EQ(socket.get() == NULL, result != OK);
     delete job;
     have_result_ = true;
     if (waiting_for_result_)
       MessageLoop::current()->Quit();
   }
 
   int WaitForResult() {
     DCHECK(!waiting_for_result_);
     while (!have_result_) {
       waiting_for_result_ = true;
@@ skipping 1466 matching lines @@
 
   pool_->CleanupTimedOutIdleSockets();
   rv = InitHandle(req.handle(), "a", 0, &req, pool_.get(), NULL);
   EXPECT_EQ(OK, rv);
   EXPECT_TRUE(req.handle()->is_reused());
 }
 
 }  // namespace
 
 }  // namespace net