Have the RequestCoordinator generate the offline_id

chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.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/offline_page_tab_helper.h"
 
 #include <memory>
 
 #include "base/bind.h"
 #include "base/feature_list.h"
@@ skipping 157 matching lines @@
   OfflinePageModelFactory::GetInstance()->SetTestingFactoryAndUse(
       browser_context(), BuildTestOfflinePageModel);
   RunUntilIdle();
 
   // Saves an offline page.
   OfflinePageModel* model =
       OfflinePageModelFactory::GetForBrowserContext(browser_context());
   std::unique_ptr<OfflinePageTestArchiver> archiver(BuildArchiver(
       kTestPageUrl, base::FilePath(FILE_PATH_LITERAL("page1.mhtml"))));
   model->SavePage(
-      kTestPageUrl, kTestClientId, std::move(archiver),
+      kTestPageUrl, kTestClientId, 0ul, std::move(archiver),
       base::Bind(&OfflinePageTabHelperTest::OnSavePageDone, AsWeakPtr()));
   RunUntilIdle();
 }
 
 void OfflinePageTabHelperTest::TearDown() {
   content::RenderViewHostTestHarness::TearDown();
 }
 
 void OfflinePageTabHelperTest::RunUntilIdle() {
   base::RunLoop().RunUntilIdle();
@@ skipping 150 matching lines @@
 TEST_F(OfflinePageTabHelperTest, SelectBestPageForCurrentTab) {
   // Saves an offline page.
   OfflinePageModel* model =
       OfflinePageModelFactory::GetForBrowserContext(browser_context());
   std::unique_ptr<OfflinePageTestArchiver> archiver(BuildArchiver(
       kTestPageUrl, base::FilePath(FILE_PATH_LITERAL("page2.mhtml"))));
 
   // We expect this copy to be used later.
   ClientId client_id(kLastNNamespace, base::IntToString(kTabId));
   model->SavePage(
-      kTestPageUrl, client_id, std::move(archiver),
+      kTestPageUrl, client_id, 0ul, std::move(archiver),
       base::Bind(&OfflinePageTabHelperTest::OnSavePageDone, AsWeakPtr()));
   RunUntilIdle();
   const int64_t expected_offline_id = offline_id();
   const GURL expected_offline_url = offline_url();
 
   archiver = BuildArchiver(kTestPageUrl,
                            base::FilePath(FILE_PATH_LITERAL("page3.html")));
   client_id.id = "39";
   model->SavePage(
-      kTestPageUrl, client_id, std::move(archiver),
+      kTestPageUrl, client_id, 0ul, std::move(archiver),
       base::Bind(&OfflinePageTabHelperTest::OnSavePageDone, AsWeakPtr()));
   RunUntilIdle();
 
   SimulateHasNetworkConnectivity(false);
   StartLoad(kTestPageUrl);
   // Gives a chance to run delayed task to do redirection.
   RunUntilIdle();
 
   const OfflinePageItem* item =
       OfflinePageUtils::GetOfflinePageFromWebContents(web_contents());
@@ skipping 50 matching lines @@
   // Saves an offline page.
   OfflinePageModel* model =
       OfflinePageModelFactory::GetForBrowserContext(browser_context());
   std::unique_ptr<OfflinePageTestArchiver> archiver(BuildArchiver(
       kTestPageUrl,
       base::FilePath(FILE_PATH_LITERAL("AsyncLoadedPage.mhtml"))));
 
   // We expect this Async Loading Namespace copy to be used.
   ClientId client_id(kAsyncNamespace, base::IntToString(kTabId));
   model->SavePage(
-      kTestPageUrl, client_id, std::move(archiver),
+      kTestPageUrl, client_id, 0ul, std::move(archiver),
       base::Bind(&OfflinePageTabHelperTest::OnSavePageDone, AsWeakPtr()));
   RunUntilIdle();
   const int64_t expected_offline_id = offline_id();
   const GURL expected_offline_url = offline_url();
 
   SimulateHasNetworkConnectivity(false);
   StartLoad(kTestPageUrl);
   // Gives a chance to run delayed task to do redirection.
   RunUntilIdle();
 
   const OfflinePageItem* item =
       OfflinePageUtils::GetOfflinePageFromWebContents(web_contents());
   EXPECT_EQ(expected_offline_id, item->offline_id);
   EXPECT_EQ(expected_offline_url, item->GetOfflineURL());
   EXPECT_EQ(kAsyncNamespace, item->client_id.name_space);
 }
 
 }  // namespace offline_pages