Add support to URLRequest for deferring redirects....

net/url_request/url_request.cc

 // Copyright (c) 2006-2009 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.h"
 
 #include "base/message_loop.h"
 #include "base/process_util.h"
 #include "base/singleton.h"
 #include "base/stats_counters.h"
@@ skipping 234 matching lines @@
   DCHECK(!is_pending_);
   DCHECK(!job_);
 
   job_ = job;
   job_->SetExtraRequestHeaders(extra_request_headers_);
 
   if (upload_.get())
     job_->SetUpload(upload_.get());
 
   is_pending_ = true;
+
   response_info_.request_time = Time::Now();
   response_info_.was_cached = false;
 
   // Don't allow errors to be sent from within Start().
   // TODO(brettw) this may cause NotifyDone to be sent synchronously,
   // we probably don't want this: they should be sent asynchronously so
   // the caller does not get reentered.
   job_->Start();
 }
 
 void URLRequest::Restart() {
   // Should only be called if the original job didn't make any progress.
   DCHECK(job_ && !job_->has_response_started());
   RestartWithJob(GetJobManager()->CreateJob(this));
 }
 
 void URLRequest::RestartWithJob(URLRequestJob *job) {
   DCHECK(job->request() == this);
+  job_->Kill();
   OrphanJob();
   status_ = URLRequestStatus();
   is_pending_ = false;
   StartJob(job);
 }
 
 void URLRequest::Cancel() {
   DoCancel(net::ERR_ABORTED, net::SSLInfo());
 }
 
@@ skipping 45 matching lines @@
 
   // Once the request fails or is cancelled, read will just return 0 bytes
   // to indicate end of stream.
   if (!status_.is_success()) {
     return true;
   }
 
   return job_->Read(dest, dest_size, bytes_read);
 }
 
-void URLRequest::ReceivedRedirect(const GURL& location) {
+void URLRequest::ReceivedRedirect(const GURL& location, bool* defer_redirect) {
   URLRequestJob* job = GetJobManager()->MaybeInterceptRedirect(this, location);
   if (job) {
     RestartWithJob(job);
   } else if (delegate_) {
-    delegate_->OnReceivedRedirect(this, location);
+    delegate_->OnReceivedRedirect(this, location, defer_redirect);
   }
 }
 
 void URLRequest::ResponseStarted() {
   URLRequestJob* job = GetJobManager()->MaybeInterceptResponse(this);
   if (job) {
     RestartWithJob(job);
   } else if (delegate_) {
     delegate_->OnResponseStarted(this);
   }
 }
 
+void URLRequest::FollowDeferredRedirect() {
+  DCHECK(job_);
+
+  job_->FollowDeferredRedirect();
+}
+
 void URLRequest::SetAuth(const wstring& username, const wstring& password) {
   DCHECK(job_);
   DCHECK(job_->NeedsAuth());
 
   job_->SetAuth(username, password);
 }
 
 void URLRequest::CancelAuth() {
   DCHECK(job_);
   DCHECK(job_->NeedsAuth());
@@ skipping 50 matching lines @@
     // following a 302 redirect, normal browsers don't do that.  Instead, they
     // all convert a POST into a GET in response to a 302 and so shall we.  For
     // 307 redirects, browsers preserve the method.  The RFC says to prompt the
     // user to confirm the generation of a new POST request, but IE omits this
     // prompt and so shall we.
     strip_post_specific_headers = method_ == "POST";
     method_ = "GET";
   }
   url_ = location;
   upload_ = NULL;
-  status_ = URLRequestStatus();
   --redirect_limit_;
 
   if (strip_post_specific_headers) {
     // If being switched from POST to GET, must remove headers that were
     // specific to the POST and don't have meaning in GET. For example
     // the inclusion of a multipart Content-Type header in GET can cause
     // problems with some servers:
     // http://code.google.com/p/chromium/issues/detail?id=843
     //
     // TODO(eroman): It would be better if this data was structured into
     // specific fields/flags, rather than a stew of extra headers.
     extra_request_headers_ = StripPostSpecificHeaders(extra_request_headers_);
   }
 
   if (!final_upload_progress_) {
     final_upload_progress_ = job_->GetUploadProgress();
   }
 
+  job_->Kill();
   OrphanJob();
 
+  status_ = URLRequestStatus();
   is_pending_ = false;
   Start();
   return net::OK;
 }
 
 URLRequestContext* URLRequest::context() {
   return context_.get();
 }
 
 void URLRequest::set_context(URLRequestContext* context) {
@@ skipping 20 matching lines @@
 }
 
 #ifndef NDEBUG
 
 URLRequestMetrics::~URLRequestMetrics() {
   DLOG_IF(WARNING, object_count != 0) <<
     "Leaking " << object_count << " URLRequest object(s)";
 }
 
 #endif