Commit 4b1a7934 authored by rch's avatar rch Committed by Commit bot

When QUIC sessions require confirmation from the server, allow either a

REJ or an SHLO to count. This should mean that anytime confirmation is
required, the handshake should be at most 1 effective RTT whereas
currently they can be 2 RTT.

Review URL: https://codereview.chromium.org/656303004

Cr-Commit-Position: refs/heads/master@{#300204}
parent 568dc714
...@@ -563,7 +563,8 @@ void QuicClientSession::OnClosedStream() { ...@@ -563,7 +563,8 @@ void QuicClientSession::OnClosedStream() {
void QuicClientSession::OnCryptoHandshakeEvent(CryptoHandshakeEvent event) { void QuicClientSession::OnCryptoHandshakeEvent(CryptoHandshakeEvent event) {
if (!callback_.is_null() && if (!callback_.is_null() &&
(!require_confirmation_ || event == HANDSHAKE_CONFIRMED)) { (!require_confirmation_ ||
event == HANDSHAKE_CONFIRMED || event == ENCRYPTION_REESTABLISHED)) {
// TODO(rtenneti): Currently for all CryptoHandshakeEvent events, callback_ // TODO(rtenneti): Currently for all CryptoHandshakeEvent events, callback_
// could be called because there are no error events in CryptoHandshakeEvent // could be called because there are no error events in CryptoHandshakeEvent
// enum. If error events are added to CryptoHandshakeEvent, then the // enum. If error events are added to CryptoHandshakeEvent, then the
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment