 Chromium Code Reviews
 Chromium Code Reviews Issue 23279011:
  Require handshake confirmation until a QUIC connection is created succesfully when using a new netw…  (Closed) 
  Base URL: svn://svn.chromium.org/chrome/trunk/src
    
  
    Issue 23279011:
  Require handshake confirmation until a QUIC connection is created succesfully when using a new netw…  (Closed) 
  Base URL: svn://svn.chromium.org/chrome/trunk/src| Index: net/quic/test_tools/mock_crypto_client_stream.cc | 
| diff --git a/net/quic/test_tools/mock_crypto_client_stream.cc b/net/quic/test_tools/mock_crypto_client_stream.cc | 
| index 79c335317144862d20d68bd646a8b7f84d7cb835..bfeb06f06292d450b6e52133dac61be187a183d6 100644 | 
| --- a/net/quic/test_tools/mock_crypto_client_stream.cc | 
| +++ b/net/quic/test_tools/mock_crypto_client_stream.cc | 
| @@ -51,6 +51,16 @@ bool MockCryptoClientStream::CryptoConnect() { | 
| return true; | 
| } | 
| +void MockCryptoClientStream::SendOnCryptoHandshakeEvent( | 
| + QuicSession::CryptoHandshakeEvent event) { | 
| + encryption_established_ = true; | 
| 
wtc
2013/08/21 01:09:32
Nit: it seems that we only need to set encryption_
 | 
| + if (event == QuicSession::HANDSHAKE_CONFIRMED) { | 
| + handshake_confirmed_ = true; | 
| + SetConfigNegotiated(); | 
| + } | 
| + session()->OnCryptoHandshakeEvent(event); | 
| +} | 
| + | 
| void MockCryptoClientStream::SetConfigNegotiated() { | 
| ASSERT_FALSE(session()->config()->negotiated()); | 
| QuicTagVector cgst; |