Commit 0e554b21 authored by Bence Béky's avatar Bence Béky Committed by Commit Bot

Add CHECK in Job::SetSpdyHttpStreamOrBidirectionalStreamImpl().

Bug: 819101
Change-Id: I6618354362d7454ad4ffa69d383ef739895932d4
Reviewed-on: https://chromium-review.googlesource.com/990512
Commit-Queue: Bence Béky <bnc@chromium.org>
Reviewed-by: default avatarRyan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547694}
parent 5d333ea7
...@@ -1156,6 +1156,10 @@ int HttpStreamFactoryImpl::Job::SetSpdyHttpStreamOrBidirectionalStreamImpl( ...@@ -1156,6 +1156,10 @@ int HttpStreamFactoryImpl::Job::SetSpdyHttpStreamOrBidirectionalStreamImpl(
DCHECK(delegate_->websocket_handshake_stream_create_helper()); DCHECK(delegate_->websocket_handshake_stream_create_helper());
if (!try_websocket_over_http2_) { if (!try_websocket_over_http2_) {
// TODO(bnc): Change to DCHECK once https://crbug.com/819101 is fixed.
CHECK(origin_url_.SchemeIs(url::kWsScheme));
CHECK(proxy_info_.is_https());
// Plaintext WebSocket is not supported over HTTP/2 proxy, // Plaintext WebSocket is not supported over HTTP/2 proxy,
// see https://crbug.com/684681. // see https://crbug.com/684681.
return ERR_NOT_IMPLEMENTED; return ERR_NOT_IMPLEMENTED;
......
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