If an idle socket has received data unexpectedly, we...

net/base/ssl_client_socket_unittest.cc

 // Copyright (c) 2006-2008 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 "base/ref_counted.h"
 #include "net/base/address_list.h"
 #include "net/base/client_socket_factory.h"
 #include "net/base/host_resolver.h"
 #include "net/base/host_resolver_unittest.h"
 #include "net/base/net_errors.h"
@@ skipping 148 matching lines @@
 
   // The Windows code happens to keep the connection
   // open now in spite of an error.  The designers of
   // this API intended to also allow the connection
   // to be closed on error, in which case the caller
   // should call ReconnectIgnoringLastError, but
   // that's currently unimplemented.
   EXPECT_TRUE(sock->IsConnected());
 }
 
+// TODO(wtc): Add unit tests for IsConnectedAndIdle:
+//   - Server closes an SSL connection (with a close_notify alert message).
+//   - Server closes the underlying TCP connection directly.
+//   - Server sends data unexpectedly.
+
 TEST_F(SSLClientSocketTest, MAYBE_Read) {
   StartOKServer();
 
   net::AddressList addr;
   net::HostResolver resolver;
   TestCompletionCallback callback;
 
   int rv = resolver.Resolve(server_.kHostName, server_.kOKHTTPSPort,
                             &addr, &callback);
   EXPECT_EQ(net::ERR_IO_PENDING, rv);
@@ skipping 120 matching lines @@
   char buf[512];
   rv = sock->Read(buf, sizeof(buf), &callback);
   EXPECT_TRUE(rv >= 0 || rv == net::ERR_IO_PENDING);
 
   if (rv == net::ERR_IO_PENDING)
     rv = callback.WaitForResult();
 
   EXPECT_NE(rv, 0);
 }