Commit 5ac4c912 authored by Ramin Halavati's avatar Ramin Halavati Committed by Commit Bot

Add traffic annotation to Websocket HTTP2 Handshake.

Network traffic annotation is passed from HttpRequestInfo to
Websocket HTTP2 Handshake.

Bug: 656607
Change-Id: Ib8c40a734ec598346457a32d75f3e2caeb8cbad0
Reviewed-on: https://chromium-review.googlesource.com/959008
Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
Reviewed-by: default avatarAdam Rice <ricea@chromium.org>
Reviewed-by: default avatarBence Béky <bnc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#542872}
parent b3ea30db
......@@ -68,6 +68,7 @@ int WebSocketHttp2HandshakeStream::InitializeStream(
RequestPriority priority,
const NetLogWithSource& net_log,
CompletionOnceCallback callback) {
DCHECK(request_info->traffic_annotation.is_valid());
request_info_ = request_info;
priority_ = priority;
net_log_ = net_log;
......@@ -117,13 +118,12 @@ int WebSocketHttp2HandshakeStream::SendRequest(
callback_ = std::move(callback);
spdy_stream_request_ = std::make_unique<SpdyStreamRequest>();
// TODO(https://crbug.com/656607): Add proper annotation here.
int rv = spdy_stream_request_->StartRequest(
SPDY_BIDIRECTIONAL_STREAM, session_, request_info_->url, priority_,
request_info_->socket_tag, net_log_,
base::BindOnce(&WebSocketHttp2HandshakeStream::StartRequestCallback,
base::Unretained(this)),
NO_TRAFFIC_ANNOTATION_BUG_656607);
NetworkTrafficAnnotationTag(request_info_->traffic_annotation));
if (rv == OK) {
StartRequestCallback(rv);
return ERR_IO_PENDING;
......
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