Track original URL when downloading a redirected page

chrome/browser/android/offline_pages/offline_page_utils.cc

 // Copyright 2015 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/offline_page_utils.h"
 
 #include "base/bind.h"
 #include "base/guid.h"
 #include "base/location.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 10 matching lines @@
 #include "chrome/browser/net/net_error_tab_helper.h"
 #include "components/offline_pages/core/background/request_coordinator.h"
 #include "components/offline_pages/core/background/save_page_request.h"
 #include "components/offline_pages/core/client_namespace_constants.h"
 #include "components/offline_pages/core/client_policy_controller.h"
 #include "components/offline_pages/core/offline_page_feature.h"
 #include "components/offline_pages/core/offline_page_item.h"
 #include "components/offline_pages/core/offline_page_model.h"
 #include "components/offline_pages/core/request_header/offline_page_header.h"
 #include "content/public/browser/browser_context.h"
+#include "content/public/browser/navigation_entry.h"
 #include "content/public/browser/web_contents.h"
-#include "url/gurl.h"
 
 namespace offline_pages {
 namespace {
 
 void OnGetPagesByURLDone(
     const GURL& url,
     int tab_id,
     const std::vector<std::string>& namespaces_to_show_in_original_tab,
     const base::Callback<void(const OfflinePageItem*)>& callback,
     const MultipleOfflinePageItemResult& pages) {
@@ skipping 193 matching lines @@
     content::BrowserContext* context,
     const GURL& url) {
   RequestCoordinator* request_coordinator =
       RequestCoordinatorFactory::GetForBrowserContext(context);
 
   // TODO(dimich): Enable in Incognito when Android Downloads implement
   // Incognito story.
   if (!request_coordinator)
     return;
 
-  ClientId client_id(kDownloadNamespace, base::GenerateGUID());
-  request_coordinator->SavePageLater(
-      url, client_id, true,
-      RequestCoordinator::RequestAvailability::ENABLED_FOR_OFFLINER);
+  RequestCoordinator::SavePageLaterParams params;
+  params.url = url;
+  params.client_id = ClientId(kDownloadNamespace, base::GenerateGUID());
+  params.user_requested = true;

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

are you missing availability here?

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

ditto

+  request_coordinator->SavePageLater(params);
 
   android::OfflinePageNotificationBridge notification_bridge;
   notification_bridge.ShowDownloadingToast();
 }
 
+// static
+GURL OfflinePageUtils::GetOriginalURLFromWebContents(
+    content::WebContents* web_contents) {
+  content::NavigationEntry* entry =
+      web_contents->GetController().GetLastCommittedEntry();
+  if (!entry || entry->GetRedirectChain().size() <= 1)
+    return GURL();
+  return entry->GetRedirectChain().front();
+}
+
 }  // namespace offline_pages