Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(183)

Unified Diff: net/websockets/websocket_basic_handshake_stream.cc

Issue 336263005: Map WebSocket URL schemes to HTTP URL schemes for auth purposes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Replace accidentally removed #include <string> Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: net/websockets/websocket_basic_handshake_stream.cc
diff --git a/net/websockets/websocket_basic_handshake_stream.cc b/net/websockets/websocket_basic_handshake_stream.cc
index f61885bac176da8bc7aea832021f6ccbcd2e306c..2eee942478e92be827f9163c76c96a0d2ee9f947 100644
--- a/net/websockets/websocket_basic_handshake_stream.cc
+++ b/net/websockets/websocket_basic_handshake_stream.cc
@@ -539,20 +539,11 @@ void WebSocketBasicHandshakeStream::ReadResponseHeadersCallback(
}
void WebSocketBasicHandshakeStream::OnFinishOpeningHandshake() {
- DCHECK(connect_delegate_);
DCHECK(http_response_info_);
- scoped_refptr<HttpResponseHeaders> headers = http_response_info_->headers;
- // If the headers are too large, HttpStreamParser will just not parse them at
- // all.
- if (headers) {
- scoped_ptr<WebSocketHandshakeResponseInfo> response(
- new WebSocketHandshakeResponseInfo(url_,
- headers->response_code(),
- headers->GetStatusText(),
- headers,
- http_response_info_->response_time));
- connect_delegate_->OnFinishOpeningHandshake(response.Pass());
- }
+ WebSocketDispatchOnFinishOpeningHandshake(connect_delegate_,
+ url_,
+ http_response_info_->headers,
+ http_response_info_->response_time);
}
int WebSocketBasicHandshakeStream::ValidateResponse(int rv) {

Powered by Google App Engine
This is Rietveld 408576698