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

Unified Diff: net/http/http_stream_factory_impl_job.cc

Issue 2621983004: Improve HttpStreamFactory NetLog events (Closed)
Patch Set: Created 3 years, 11 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/http/http_stream_factory_impl_job.cc
diff --git a/net/http/http_stream_factory_impl_job.cc b/net/http/http_stream_factory_impl_job.cc
index 2cd3bcbe0b6c7cf73b0129896405537a9a92cfcb..70bb421be931bb86549ed6b145c1f0338f25722a 100644
--- a/net/http/http_stream_factory_impl_job.cc
+++ b/net/http/http_stream_factory_impl_job.cc
@@ -685,15 +685,13 @@ int HttpStreamFactoryImpl::Job::StartInternal() {
int HttpStreamFactoryImpl::Job::DoStart() {
const NetLogWithSource* net_log = delegate_->GetNetLog(this);
- if (net_log) {
- net_log_.BeginEvent(
- NetLogEventType::HTTP_STREAM_JOB,
- base::Bind(&NetLogHttpStreamJobCallback, net_log->source(),
- &request_info_.url, &origin_url_, &alternative_service_,
- priority_));
- net_log->AddEvent(NetLogEventType::HTTP_STREAM_REQUEST_STARTED_JOB,
- net_log_.source().ToEventParametersCallback());
- }
+ net_log_.BeginEvent(
+ NetLogEventType::HTTP_STREAM_JOB,
+ base::Bind(&NetLogHttpStreamJobCallback, net_log->source(),
+ &request_info_.url, &origin_url_, &alternative_service_,
+ priority_));
+ net_log->AddEvent(NetLogEventType::HTTP_STREAM_REQUEST_STARTED_JOB,
+ net_log_.source().ToEventParametersCallback());
// Don't connect to restricted ports.
if (!IsPortAllowedForScheme(destination_.port(),

Powered by Google App Engine
This is Rietveld 408576698