Refactor the offline page storage to include client namespace and id.

components/offline_pages/offline_page_test_store.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 "components/offline_pages/offline_page_test_store.h"
 
 #include "base/bind.h"
 #include "base/location.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 22 matching lines @@
   task_runner_->PostTask(FROM_HERE,
                          base::Bind(callback, load_status, GetAllPages()));
 }
 
 void OfflinePageTestStore::AddOrUpdateOfflinePage(
     const OfflinePageItem& offline_page,
     const UpdateCallback& callback) {
   last_saved_page_ = offline_page;
   bool result = scenario_ != TestScenario::WRITE_FAILED;
   if (result)
-    offline_pages_[offline_page.bookmark_id] = offline_page;
+    offline_pages_[offline_page.offline_id] = offline_page;
   if (!callback.is_null())
     task_runner_->PostTask(FROM_HERE, base::Bind(callback, result));
 }
 
 void OfflinePageTestStore::RemoveOfflinePages(
-    const std::vector<int64_t>& bookmark_ids,
+    const std::vector<int64_t>& offline_ids,
     const UpdateCallback& callback) {
-  ASSERT_FALSE(bookmark_ids.empty());
+  ASSERT_FALSE(offline_ids.empty());
   bool result = false;
   if (scenario_ != TestScenario::REMOVE_FAILED) {
-    for (const auto& bookmark_id : bookmark_ids) {
-      auto iter = offline_pages_.find(bookmark_id);
+    for (const auto& offline_id : offline_ids) {
+      auto iter = offline_pages_.find(offline_id);
       if (iter != offline_pages_.end()) {
         offline_pages_.erase(iter);
         result = true;
       }
     }
   }
 
   task_runner_->PostTask(FROM_HERE, base::Bind(callback, result));
 }
 
 void OfflinePageTestStore::Reset(const ResetCallback& callback) {
   offline_pages_.clear();
   task_runner_->PostTask(FROM_HERE, base::Bind(callback, true));
 }
 
 void OfflinePageTestStore::UpdateLastAccessTime(
-    int64_t bookmark_id,
+    int64_t offline_id,
     const base::Time& last_access_time) {
-  auto iter = offline_pages_.find(bookmark_id);
+  auto iter = offline_pages_.find(offline_id);
   if (iter == offline_pages_.end())
     return;
   iter->second.last_access_time = last_access_time;
 }
 
 std::vector<OfflinePageItem> OfflinePageTestStore::GetAllPages() const {
   std::vector<OfflinePageItem> offline_pages;
   for (const auto& id_page_pair : offline_pages_)
     offline_pages.push_back(id_page_pair.second);
   return offline_pages;
 }
 
 }  // namespace offline_pages