Have the RequestCoordinator generate the offline_id

chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.cc

Index: chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.cc
diff --git a/chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.cc b/chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.cc
index c7392a3ff8f8105f82c8bdb761686e59f8ad3e50..0702e5093a4d05f0d8bf5a47aac998190bd0bd3e 100644
--- a/chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.cc
+++ b/chrome/browser/android/offline_pages/offline_page_tab_helper_unittest.cc
@@ -172,7 +172,7 @@ void OfflinePageTabHelperTest::SetUp() {
   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();
 }
@@ -343,7 +343,7 @@ TEST_F(OfflinePageTabHelperTest, SelectBestPageForCurrentTab) {
   // 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();
@@ -353,7 +353,7 @@ TEST_F(OfflinePageTabHelperTest, SelectBestPageForCurrentTab) {
                            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();
 
@@ -424,7 +424,7 @@ TEST_F(OfflinePageTabHelperTest, SwitchToOfflineAsyncLoadedPageOnNoNetwork) {
   // 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();