Track original URL when downloading a redirected page

chrome/browser/android/offline_pages/background_loader_offliner.cc

 // Copyright 2016 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 "chrome/browser/android/offline_pages/background_loader_offliner.h"
 
 #include "base/metrics/histogram_macros.h"
 #include "base/sys_info.h"
 #include "chrome/browser/android/offline_pages/offline_page_mhtml_archiver.h"
 #include "chrome/browser/android/offline_pages/offliner_helper.h"
@@ skipping 135 matching lines @@
       content::WebContentsObserver::web_contents());
 
   std::unique_ptr<OfflinePageArchiver> archiver(
       new OfflinePageMHTMLArchiver(web_contents));
 
   OfflinePageModel::SavePageParams params;
   params.url = web_contents->GetLastCommittedURL();
   params.client_id = request.client_id();
   params.proposed_offline_id = request.request_id();
   params.is_background = true;
+
+  // Pass in the original URL if it is different from the last committed URL
+  // when redirects occur.

[fgorski, 2017/02/27 17:34:35]

please sync. I think Cathy was making a similar change recently.
It may make sense to add Cathy to this review.

[jianli, 2017/02/27 23:20:06]

Synced. Cathy ported my previous original URL change from prerender.

+  if (!request.original_url().is_empty())
+    params.original_url = request.original_url();
+  else if (params.url != request.url())
+    params.original_url = request.url();
+
   offline_page_model_->SavePage(
       params, std::move(archiver),
       base::Bind(&BackgroundLoaderOffliner::OnPageSaved,
                  weak_ptr_factory_.GetWeakPtr()));
 }
 
 void BackgroundLoaderOffliner::RenderProcessGone(
     base::TerminationStatus status) {
   if (pending_request_) {
     SavePageRequest request(*pending_request_.get());
@@ skipping 103 matching lines @@
       application_state ==
           base::android::APPLICATION_STATE_HAS_RUNNING_ACTIVITIES) {
     DVLOG(1) << "App became active, canceling current offlining request";
     SavePageRequest* request = pending_request_.get();
     Cancel();
     completion_callback_.Run(*request, RequestStatus::FOREGROUND_CANCELED);
   }
 }
 
 }  // namespace offline_pages