Track original URL when downloading a redirected page

chrome/browser/ui/webui/offline/offline_internals_ui_message_handler.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/ui/webui/offline/offline_internals_ui_message_handler.h"
 
 #include <stdint.h>
 #include <stdlib.h>
 #include <vector>
 
@@ skipping 149 matching lines @@
       save_page_requests.Append(save_page_request);
       save_page_request->SetString("onlineUrl", request->url().spec());
       save_page_request->SetDouble("creationTime",
                                    request->creation_time().ToJsTime());
       save_page_request->SetString("status", GetStringFromSavePageStatus());
       save_page_request->SetString("namespace",
                                    request->client_id().name_space);
       save_page_request->SetDouble("lastAttempt",
                                    request->last_attempt_time().ToJsTime());
       save_page_request->SetString("id", std::to_string(request->request_id()));
+      save_page_request->SetString("originalUrl",
+                                   request->original_url().spec());
     }
   }
   ResolveJavascriptCallback(base::StringValue(callback_id), save_page_requests);
 }
 
 void OfflineInternalsUIMessageHandler::HandleGetRequestQueue(
     const base::ListValue* args) {
   AllowJavascript();
   std::string callback_id;
   CHECK(args->GetString(0, &callback_id));
@@ skipping 95 matching lines @@
 
   if (request_coordinator_) {
     std::string url;
     CHECK(args->GetString(1, &url));
 
     // To be visible in Downloads UI, these items need a well-formed GUID
     // and AsyncNamespace in their ClientId.
     std::ostringstream id_stream;
     id_stream << base::GenerateGUID();
 
+    offline_pages::RequestCoordinator::SavePageLaterParams params;
+    params.url = GURL(url);
+    params.client_id = offline_pages::ClientId(offline_pages::kAsyncNamespace,
+                                               id_stream.str());
+    params.user_requested = true;

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

availability?

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

ditto.

     ResolveJavascriptCallback(
-        *callback_id,
-        base::FundamentalValue(request_coordinator_->SavePageLater(
-                GURL(url), offline_pages::ClientId(
-                               offline_pages::kAsyncNamespace, id_stream.str()),
-                true, offline_pages::RequestCoordinator::RequestAvailability::
-                          ENABLED_FOR_OFFLINER) > 0));
+        *callback_id, base::FundamentalValue(
+                          request_coordinator_->SavePageLater(params) > 0));
   } else {
     ResolveJavascriptCallback(*callback_id, base::FundamentalValue(false));
   }
 }
 
 void OfflineInternalsUIMessageHandler::RegisterMessages() {
   web_ui()->RegisterMessageCallback(
       "deleteSelectedPages",
       base::Bind(&OfflineInternalsUIMessageHandler::HandleDeleteSelectedPages,
                  weak_ptr_factory_.GetWeakPtr()));
@@ skipping 37 matching lines @@
 
   // Get the offline page model associated with this web ui.
   Profile* profile = Profile::FromWebUI(web_ui());
   offline_page_model_ =
       offline_pages::OfflinePageModelFactory::GetForBrowserContext(profile);
   request_coordinator_ =
       offline_pages::RequestCoordinatorFactory::GetForBrowserContext(profile);
 }
 
 }  // namespace offline_internals