If an idle socket has received data unexpectedly, we can't

reuse it.

Add the IsConnectedAndIdle method, which returns true if the
connection is still alive and idle (hasn't received any data
unexpectedly).

R=eroman
BUG=4606
Review URL: http://codereview.chromium.org/21501

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

1 files changed, 11 insertions, 0 deletions

diff --git a/net/base/ssl_client_socket_win.cc b/net/base/ssl_client_socket_win.cc
index 5f73746..6aba9c2 100644
--- a/net/base/ssl_client_socket_win.cc
+++ b/net/base/ssl_client_socket_win.cc
@@ -314,6 +314,17 @@ bool SSLClientSocketWin::IsConnected() const {
   return completed_handshake_ && transport_->IsConnected();
 }
 
+bool SSLClientSocketWin::IsConnectedAndIdle() const {
+  // Unlike IsConnected, this method doesn't return a false positive.
+  //
+  // Strictly speaking, we should check if we have received the close_notify
+  // alert message from the server, and return false in that case.  Although
+  // the close_notify alert message means EOF in the SSL layer, it is just
+  // bytes to the transport layer below, so transport_->IsConnectedAndIdle()
+  // returns the desired false when we receive close_notify.
+  return completed_handshake_ && transport_->IsConnectedAndIdle();
+}
+
 int SSLClientSocketWin::Read(char* buf, int buf_len,
                              CompletionCallback* callback) {
   DCHECK(completed_handshake_);