Fix a race condition in the unit test. The MockPending jobs actually

induce a slight delay between the time the connect is called and when
the posted task can complete.  We weren't waiting for the 2ms to
elapse before expecting that it had completed.

BUG=47836
TEST=self

Review URL: http://codereview.chromium.org/2893010

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@52101 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 8 insertions, 1 deletions

diff --git a/net/socket/client_socket_pool_base_unittest.cc b/net/socket/client_socket_pool_base_unittest.cc
index a936c8b..03c7879 100644
--- a/net/socket/client_socket_pool_base_unittest.cc
+++ b/net/socket/client_socket_pool_base_unittest.cc
@@ -117,6 +117,10 @@ class TestConnectJob : public ConnectJob {
     kMockAdvancingLoadStateJob,
   };
 
+  // The kMockPendingJob uses a slight delay before allowing the connect
+  // to complete.
+  static const int kPendingConnectDelay = 2;
+
   TestConnectJob(JobType job_type,
                  const std::string& group_name,
                  const TestClientSocketPoolBase::Request& request,
@@ -169,7 +173,7 @@ class TestConnectJob : public ConnectJob {
                 &TestConnectJob::DoConnect,
                 true /* successful */,
                 true /* async */),
-            2);
+            kPendingConnectDelay);
         return ERR_IO_PENDING;
       case kMockPendingFailingJob:
         set_load_state(LOAD_STATE_CONNECTING);
@@ -884,6 +888,9 @@ TEST_F(ClientSocketPoolBaseTest, CancelPendingSocketAtSocketLimit) {
     // Cancel the stalled request.
     handles[0].Reset();
 
+    // Wait for the pending job to be guaranteed to complete.
+    PlatformThread::Sleep(TestConnectJob::kPendingConnectDelay * 2);
+
     MessageLoop::current()->RunAllPending();
 
     // Now we should have a connect job.