Track original URL when downloading a redirected page

chrome/browser/android/offline_pages/offline_page_utils_unittest.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 <stdint.h>
 #include <utility>
 
 #include "base/callback.h"
@@ skipping 168 matching lines @@
   archiver = BuildArchiver(kTestPage2Url,
                            base::FilePath(FILE_PATH_LITERAL("page2.mhtml")));
   client_id.id = kTestPage2ClientId;
   SavePage(kTestPage2Url, client_id, std::move(archiver));
 }
 
 void OfflinePageUtilsTest::CreateRequests() {
   RequestCoordinator* request_coordinator =
       RequestCoordinatorFactory::GetForBrowserContext(profile());
 
-  offline_pages::ClientId client_id;
-  client_id.name_space = kDownloadNamespace;
-  client_id.id = kTestPage3ClientId;
-  request_coordinator->SavePageLater(
-      kTestPage3Url, client_id, true,
-      RequestCoordinator::RequestAvailability::ENABLED_FOR_OFFLINER);
+  RequestCoordinator::SavePageLaterParams params;
+  params.url = kTestPage3Url;
+  params.client_id =
+      offline_pages::ClientId(kDownloadNamespace, kTestPage3ClientId);

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

are you missing availability here?

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

ditto.

+  request_coordinator->SavePageLater(params);
   RunUntilIdle();
 }
 
 std::unique_ptr<OfflinePageTestArchiver> OfflinePageUtilsTest::BuildArchiver(
     const GURL& url,
     const base::FilePath& file_name) {
   std::unique_ptr<OfflinePageTestArchiver> archiver(new OfflinePageTestArchiver(
       this, url, OfflinePageArchiver::ArchiverResult::SUCCESSFULLY_CREATED,
       base::string16(), kTestFileSize, base::ThreadTaskRunnerHandle::Get()));
   archiver->set_filename(file_name);
@@ skipping 49 matching lines @@
       GURL("http://example.com/"), GURL("http://example.com/#test")));
   EXPECT_TRUE(OfflinePageUtils::EqualsIgnoringFragment(
       GURL("http://example.com/#test"), GURL("http://example.com/")));
   EXPECT_TRUE(OfflinePageUtils::EqualsIgnoringFragment(
       GURL("http://example.com/#test"), GURL("http://example.com/#test2")));
   EXPECT_FALSE(OfflinePageUtils::EqualsIgnoringFragment(
       GURL("http://example.com/"), GURL("http://test.com/#test")));
 }
 
 }  // namespace offline_pages