Prevent URLRequestRedirectJob from doing async execution

net/url_request/url_request_redirect_job.cc

Index: net/url_request/url_request_redirect_job.cc
diff --git a/net/url_request/url_request_redirect_job.cc b/net/url_request/url_request_redirect_job.cc
index b127220552e2975126e72b366cba95f49d42d33b..407473f4ae2699cbc59e675562f4c89a050dc282 100644
--- a/net/url_request/url_request_redirect_job.cc
+++ b/net/url_request/url_request_redirect_job.cc
@@ -81,6 +81,10 @@ int URLRequestRedirectJob::GetResponseCode() const {
 URLRequestRedirectJob::~URLRequestRedirectJob() {}
 
 void URLRequestRedirectJob::StartAsync() {
+  // By this moment request might be already canceled and this job killed
+  if (!request_ || !request_->status().is_success())
+    return;

[mmenke, 2015/07/22 19:03:56]

Per earlier comment, I'd much rather we override the Kill() method, and clear
the weak pointers there.  I think it's cleaner to cancel what we're doing when
we're cancelled, rather than to check at a later point in time if we were
cancelled in the past.

+
   receive_headers_end_ = base::TimeTicks::Now();
   response_time_ = base::Time::Now();