[Offline pages] OPM SQL store: moving load to constructor, updating read to GetOfflinePages

components/offline_pages/offline_page_model_impl.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 "components/offline_pages/offline_page_model_impl.h"
 
 #include <algorithm>
 #include <limits>
 
 #include "base/bind.h"
@@ skipping 753 matching lines @@
 
   // No need to fire OfflinePageModelChanged event since updating access info
   // should not have any impact to the UI.
 }
 
 void OfflinePageModelImpl::OnEnsureArchivesDirCreatedDone(
     const base::TimeTicks& start_time) {
   UMA_HISTOGRAM_TIMES("OfflinePages.Model.ArchiveDirCreationTime",
                       base::TimeTicks::Now() - start_time);
 
-  store_->Load(base::Bind(&OfflinePageModelImpl::OnLoadDone,
-                          weak_ptr_factory_.GetWeakPtr(), start_time));
+  store_->GetOfflinePages(base::Bind(&OfflinePageModelImpl::OnLoadDone,
+                                     weak_ptr_factory_.GetWeakPtr(),
+                                     start_time));
 }
 
 void OfflinePageModelImpl::OnLoadDone(
     const base::TimeTicks& start_time,
     OfflinePageMetadataStore::LoadStatus load_status,
     const std::vector<OfflinePageItem>& offline_pages) {
   DCHECK(!is_loaded_);
   is_loaded_ = true;
 
   // TODO(jianli): rebuild the store upon failure.
@@ skipping 211 matching lines @@
     const base::Closure& callback,
     bool success) {
   DCHECK(success);
   if (!success) {
     offline_event_logger_.RecordStoreClearError();
     UMA_HISTOGRAM_ENUMERATION("OfflinePages.ClearAllStatus2",
                               STORE_RESET_FAILED, CLEAR_ALL_STATUS_COUNT);
   }
 
   offline_pages_.clear();
-  store_->Load(base::Bind(&OfflinePageModelImpl::OnReloadStoreDoneForClearAll,
-                          weak_ptr_factory_.GetWeakPtr(), callback));
+  store_->GetOfflinePages(
+      base::Bind(&OfflinePageModelImpl::OnReloadStoreDoneForClearAll,
+                 weak_ptr_factory_.GetWeakPtr(), callback));
 }
 
 void OfflinePageModelImpl::OnReloadStoreDoneForClearAll(
     const base::Closure& callback,
     OfflinePageMetadataStore::LoadStatus load_status,
     const std::vector<OfflinePageItem>& offline_pages) {
   DCHECK_EQ(OfflinePageMetadataStore::LOAD_SUCCEEDED, load_status);
   UMA_HISTOGRAM_ENUMERATION(
       "OfflinePages.ClearAllStatus2",
       load_status == OfflinePageMetadataStore::LOAD_SUCCEEDED
@@ skipping 51 matching lines @@
 void OfflinePageModelImpl::RunWhenLoaded(const base::Closure& task) {
   if (!is_loaded_) {
     delayed_tasks_.push_back(task);
     return;
   }
 
   base::ThreadTaskRunnerHandle::Get()->PostTask(FROM_HERE, task);
 }
 
 }  // namespace offline_pages