This CL corrects a bug in which the OnHandshakeComplete callback for an ssl session was never called

net/test/spawned_test_server/base_test_server.h

Index: net/test/spawned_test_server/base_test_server.h
diff --git a/net/test/spawned_test_server/base_test_server.h b/net/test/spawned_test_server/base_test_server.h
index 6dd67db3038a186d192e2e8baee4bf509c9aded8..fb71222499c9761b060a9abd5b53400cf15a44ab 100644
--- a/net/test/spawned_test_server/base_test_server.h
+++ b/net/test/spawned_test_server/base_test_server.h
@@ -203,6 +203,10 @@ class BaseTestServer {
 
     // Whether to enable NPN support.
     bool enable_npn;
+
+    // Whether to disable TLS session caching. When session caching is
+    // disabled, the server will generate a new Session ID for every connection.

[wtc, 2014/08/07 20:04:04]

Hmm... "generate a new session ID" is not the behavior we want to test. What we
want to test is that the server returns an empty session ID in ServerHello.

I checked the tlslite/tls_connection.py code. It seems that if 'sessionCache' is
'None', it generates an empty session ID for ServerHello:

        # Create the ServerHello message
        if sessionCache:
            sessionID = getRandomBytes(32)
        else:
            sessionID = bytearray(0)

So I think this comment should be corrected.

+    bool disable_session_cache;
   };
 
   // Pass as the 'host' parameter during construction to server on 127.0.0.1