[Offline Pages] Remove load state from public OfflinePageModel API.

components/offline_pages/core/offline_page_model.h

 // 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.
 
 #ifndef COMPONENTS_OFFLINE_PAGES_CORE_OFFLINE_PAGE_MODEL_H_
 #define COMPONENTS_OFFLINE_PAGES_CORE_OFFLINE_PAGE_MODEL_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 57 matching lines @@
     // kInvalidOfflineId, a new, random ID will be generated.
     int64_t proposed_offline_id;
 
     // The original URL of the page to save. Empty if no redirect occurs.
     GURL original_url;
   };
 
   // Observer of the OfflinePageModel.
   class Observer {
    public:
-    // Invoked when the model has finished loading.
-    virtual void OfflinePageModelLoaded(OfflinePageModel* model) = 0;
-
     // Invoked when the model is being updated due to adding an offline page.
     virtual void OfflinePageAdded(OfflinePageModel* model,
                                   const OfflinePageItem& added_page) = 0;
 
     // Invoked when an offline copy related to |offline_id| was deleted.
     virtual void OfflinePageDeleted(int64_t offline_id,
                                     const ClientId& client_id) = 0;
 
    protected:
     virtual ~Observer() = default;
@@ skipping 10 matching lines @@
   static bool CanSaveURL(const GURL& url);
 
   OfflinePageModel();
   ~OfflinePageModel() override;
 
   virtual void AddObserver(Observer* observer) = 0;
   virtual void RemoveObserver(Observer* observer) = 0;
 
   static const int64_t kInvalidOfflineId = 0;
 
-  // Attempts to save a page offline per |save_page_params|. Requires that the
-  // model is loaded.  Generates a new offline id or uses the proposed offline
-  // id in |save_page_params| and returns it.
+  // Attempts to save a page offline per |save_page_params|. Generates a new
+  // offline id or uses the proposed offline id in |save_page_params| and
+  // returns it.
   virtual void SavePage(const SavePageParams& save_page_params,
                         std::unique_ptr<OfflinePageArchiver> archiver,
                         const SavePageCallback& callback) = 0;
 
   // Marks that the offline page related to the passed |offline_id| has been
   // accessed. Its access info, including last access time and access count,
-  // will be updated. Requires that the model is loaded.
+  // will be updated.
   virtual void MarkPageAccessed(int64_t offline_id) = 0;
 
   // Deletes pages based on |offline_ids|.
   virtual void DeletePagesByOfflineId(const std::vector<int64_t>& offline_ids,
                                       const DeletePageCallback& callback) = 0;
 
   // Deletes all pages associated with any of |client_ids|.
   virtual void DeletePagesByClientIds(const std::vector<ClientId>& client_ids,
                                       const DeletePageCallback& callback) = 0;
 
@@ skipping 33 matching lines @@
   // Returns the offline pages that are related to |url|. |url_search_mode|
   // controls how the url match is done. See URLSearchMode for more details.
   virtual void GetPagesByURL(
       const GURL& url,
       URLSearchMode url_search_mode,
       const MultipleOfflinePageItemCallback& callback) = 0;
 
   // Returns the policy controller.
   virtual ClientPolicyController* GetPolicyController() = 0;
 
-  // TODO(dougarnett): Remove this and its uses.
-  virtual bool is_loaded() const = 0;
-
   // Returns the logger. Ownership is retained by the model.
   virtual OfflineEventLogger* GetLogger() = 0;
 };
 
 }  // namespace offline_pages
 
 #endif  // COMPONENTS_OFFLINE_PAGES_CORE_OFFLINE_PAGE_MODEL_H_