Prevent URLRequestRedirectJob from doing async execution

net/url_request/url_request_redirect_job.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/url_request/url_request_redirect_job.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
@@ skipping 47 matching lines @@
 
 void URLRequestRedirectJob::Start() {
   request()->net_log().AddEvent(
       NetLog::TYPE_URL_REQUEST_REDIRECT_JOB,
       NetLog::StringCallback("reason", &redirect_reason_));
   base::ThreadTaskRunnerHandle::Get()->PostTask(
       FROM_HERE, base::Bind(&URLRequestRedirectJob::StartAsync,
                             weak_factory_.GetWeakPtr()));
 }
 
+void URLRequestRedirectJob::Kill() {
+  weak_factory_.InvalidateWeakPtrs();
+  URLRequestJob::Kill();
+}
+
 bool URLRequestRedirectJob::CopyFragmentOnRedirect(const GURL& location) const {
   // The instantiators have full control over the desired redirection target,
   // including the reference fragment part of the URL.
   return false;
 }
 
 int URLRequestRedirectJob::GetResponseCode() const {
   // Should only be called after the URLRequest has been notified there's header
   // information.
   DCHECK(fake_headers_.get());
   return response_code_;
 }
 
 URLRequestRedirectJob::~URLRequestRedirectJob() {}
 
 void URLRequestRedirectJob::StartAsync() {
+  DCHECK(request_);
+  DCHECK(request_->status().is_success());
+
   receive_headers_end_ = base::TimeTicks::Now();
   response_time_ = base::Time::Now();
 
   std::string header_string =
       base::StringPrintf("HTTP/1.1 %i Internal Redirect\n"
                              "Location: %s\n"
                              "Non-Authoritative-Reason: %s",
                          response_code_,
                          redirect_destination_.spec().c_str(),
                          redirect_reason_.c_str());
@@ skipping 25 matching lines @@
           base::Unretained(fake_headers_.get())));
 
   // TODO(mmenke):  Consider calling the NetworkDelegate with the headers here.
   // There's some weirdness about how to handle the case in which the delegate
   // tries to modify the redirect location, in terms of how IsSafeRedirect
   // should behave, and whether the fragment should be copied.
   URLRequestJob::NotifyHeadersComplete();
 }
 
 }  // namespace net