Plumb redirect info out of net, through content, and into child processes.

content/browser/download/download_resource_handler.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 "content/browser/download/download_resource_handler.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/logging.h"
@@ skipping 123 matching lines @@
       PowerSaveBlocker::kPowerSaveBlockPreventAppSuspension,
       "Download in progress");
 }
 
 bool DownloadResourceHandler::OnUploadProgress(uint64 position,
                                                uint64 size) {
   return true;
 }
 
 bool DownloadResourceHandler::OnRequestRedirected(
-    const GURL& url,
+    const net::RedirectInfo& redirect_info,
     ResourceResponse* response,
     bool* defer) {

[mmenke, 2014/07/17 16:55:09]

Was code you removed redundant, or should we be setting the policy somewhere for
downloads?

Hrm...Wonder if there's a test for this.  Suspect not.

[davidben, 2014/07/17 22:03:54]

Hrm, oh it's not quite redundant because requests may begin with
DownloadResourceHandler attached. I've moved it to RDH::BeginDownload. (If this
download began life at the BRH, it would never see a redirect, so that's the
only case.)

-  // We treat a download as a main frame load, and thus update the policy URL
-  // on redirects.
-  request()->set_first_party_for_cookies(url);
   return true;
 }
 
 // Send the download creation information to the download thread.
 bool DownloadResourceHandler::OnResponseStarted(
     ResourceResponse* response,
     bool* defer) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   // There can be only one (call)
   DCHECK(!on_response_started_called_);
@@ skipping 368 matching lines @@
   // tab_info_ must be destroyed on UI thread, since
   // InitializeDownloadTabInfoOnUIThread might still be using it.
   if (tab_info_)
     BrowserThread::DeleteSoon(BrowserThread::UI, FROM_HERE, tab_info_);
 
   UMA_HISTOGRAM_TIMES("SB2.DownloadDuration",
                       base::TimeTicks::Now() - download_start_time_);
 }
 
 }  // namespace content