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

Unified Diff: net/http/http_stream_factory_impl.cc

Issue 2783683002: Log source_dependency in HTTP2_SESSION_SEND_HEADERS. (Closed)
Patch Set: Fix use-after-free. Created 3 years, 9 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
« no previous file with comments | « net/http/http_stream_factory_impl.h ('k') | net/http/http_stream_factory_impl_job.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_stream_factory_impl.cc
diff --git a/net/http/http_stream_factory_impl.cc b/net/http/http_stream_factory_impl.cc
index 07978753b28b3ff94f469456f143613d3e642190..0b79168152f873d8ecf6798581f719b49853907b 100644
--- a/net/http/http_stream_factory_impl.cc
+++ b/net/http/http_stream_factory_impl.cc
@@ -225,7 +225,8 @@ void HttpStreamFactoryImpl::OnNewSpdySessionReady(
const ProxyInfo& used_proxy_info,
bool was_alpn_negotiated,
NextProto negotiated_protocol,
- bool using_spdy) {
+ bool using_spdy,
+ NetLogSource source_dependency) {
while (true) {
if (!spdy_session)
break;
@@ -249,13 +250,13 @@ void HttpStreamFactoryImpl::OnNewSpdySessionReady(
HttpStreamRequest::BIDIRECTIONAL_STREAM) {
request->OnBidirectionalStreamImplReady(
used_ssl_config, used_proxy_info,
- new BidirectionalStreamSpdyImpl(spdy_session));
+ new BidirectionalStreamSpdyImpl(spdy_session, source_dependency));
} else {
bool use_relative_url =
direct || request->url().SchemeIs(url::kHttpsScheme);
- request->OnStreamReady(
- used_ssl_config, used_proxy_info,
- new SpdyHttpStream(spdy_session, use_relative_url));
+ request->OnStreamReady(used_ssl_config, used_proxy_info,
+ new SpdyHttpStream(spdy_session, use_relative_url,
+ source_dependency));
}
}
// TODO(mbelshe): Alert other valid requests.
« no previous file with comments | « net/http/http_stream_factory_impl.h ('k') | net/http/http_stream_factory_impl_job.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698