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

 // Copyright 2013 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.
 
 #ifndef NET_TEST_SPAWNED_TEST_SERVER_BASE_TEST_SERVER_H_
 #define NET_TEST_SPAWNED_TEST_SERVER_BASE_TEST_SERVER_H_
 
 #include <string>
 #include <utility>
 #include <vector>
@@ skipping 185 matching lines @@
     // TODO(ekasper): replace with valid SCT files for test certs.
     // (Fake) SignedCertificateTimestampList (as a raw binary string) to send in
     // a TLS extension.
     std::string signed_cert_timestamps_tls_ext;
 
     // Whether to staple the OCSP response.
     bool staple_ocsp_response;
 
     // Whether to enable NPN support.
     bool enable_npn;
+
+    // Tells the server to use a session cache of None.
+    bool none_session_cache;

[Ryan Sleevi, 2014/08/06 01:31:30]

This isn't really descriptive in the context of just this header file.

// Whether to enable TLS session caching. When session caching is
// disabled, the server will generate a new Session ID for every connection.
bool enable_session_cache;

(with a default of true)

[mshelley, 2014/08/06 03:09:31]

Done.

   };
 
   // Pass as the 'host' parameter during construction to server on 127.0.0.1
   static const char kLocalhost[];
 
   // Initialize a TestServer listening on a specific host (IP or hostname).
   BaseTestServer(Type type,  const std::string& host);
 
   // Initialize a TestServer with a specific set of SSLOptions for HTTPS or WSS.
   BaseTestServer(Type type, const SSLOptions& ssl_options);
@@ skipping 102 matching lines @@
   bool ws_basic_auth_;
 
   scoped_ptr<ScopedPortException> allowed_port_;
 
   DISALLOW_COPY_AND_ASSIGN(BaseTestServer);
 };
 
 }  // namespace net
 
 #endif  // NET_TEST_SPAWNED_TEST_SERVER_BASE_TEST_SERVER_H_