Add Store+Sync to reading list.

ios/chrome/browser/reading_list/reading_list_model_impl.h

 // 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.
 
 #ifndef IOS_CHROME_BROWSER_READING_LIST_READING_LIST_MODEL_IMPL_H_
 #define IOS_CHROME_BROWSER_READING_LIST_READING_LIST_MODEL_IMPL_H_
 
 #include <memory>
 
+#include "base/threading/non_thread_safe.h"

[pavely, 2016/11/14 07:45:33]

non_thread_safe.h is not needed here. 

[Olivier, 2016/11/14 11:36:49]

Done.

 #include "components/keyed_service/core/keyed_service.h"
 #include "ios/chrome/browser/reading_list/reading_list_entry.h"
 #include "ios/chrome/browser/reading_list/reading_list_model.h"
+#include "ios/chrome/browser/reading_list/reading_list_model_storage.h"
+#include "ios/chrome/browser/reading_list/reading_list_store_delegate.h"
 
-class ReadingListModelStorage;
 class PrefService;
 
+using ReadingListEntries = std::vector<ReadingListEntry>;
+
 // Concrete implementation of a reading list model using in memory lists.
-class ReadingListModelImpl : public ReadingListModel, public KeyedService {
+class ReadingListModelImpl : public ReadingListModel,
+                             public ReadingListStoreDelegate,
+                             public KeyedService {
  public:
   // Initialize a ReadingListModelImpl to load and save data in
   // |persistence_layer|.
-  ReadingListModelImpl(std::unique_ptr<ReadingListModelStorage> storage,
+  ReadingListModelImpl(std::unique_ptr<ReadingListModelStorage> storage_layer,
                        PrefService* pref_service);
 
   // Initialize a ReadingListModelImpl without persistence. Data will not be
   // persistent across sessions.
   ReadingListModelImpl();
 
+  syncer::ModelTypeSyncBridge* GetModelTypeSyncBridge() override;
+
   ~ReadingListModelImpl() override;
 
+  void StoreLoaded(std::unique_ptr<ReadingListEntries> unread,
+                   std::unique_ptr<ReadingListEntries> read) override;
+
   // KeyedService implementation.
   void Shutdown() override;
 
   // ReadingListModel implementation.
   bool loaded() const override;
 
   size_t unread_size() const override;
   size_t read_size() const override;
 
   bool HasUnseenEntries() const override;
   void ResetUnseenEntries() override;
 
   const ReadingListEntry& GetUnreadEntryAtIndex(size_t index) const override;
   const ReadingListEntry& GetReadEntryAtIndex(size_t index) const override;
 
-  const ReadingListEntry* GetEntryFromURL(const GURL& gurl) const override;
+  const ReadingListEntry* GetEntryFromURL(const GURL& gurl,
+                                          bool* read) const override;
 
   bool CallbackEntryURL(
       const GURL& url,
       base::Callback<void(const ReadingListEntry&)> callback) const override;
 
   void RemoveEntryByURL(const GURL& url) override;
 
   // Temporary method
   void RemoveEntryByUrl(const GURL& url) override;
 
   const ReadingListEntry& AddEntry(const GURL& url,
                                    const std::string& title) override;
 
   void MarkReadByURL(const GURL& url) override;
   void MarkUnreadByURL(const GURL& url) override;
 
   void SetEntryTitle(const GURL& url, const std::string& title) override;
   void SetEntryDistilledURL(const GURL& url,
                             const GURL& distilled_url) override;
   void SetEntryDistilledState(
       const GURL& url,
       ReadingListEntry::DistillationState state) override;
 
+  void SyncAddEntry(std::unique_ptr<ReadingListEntry> entry,
+                    bool read) override;
+  void SyncRemoveEntry(const GURL& gurl) override;
+
+  std::unique_ptr<ReadingListModel::ScopedReadingListBatchUpdate>
+  CreateBatchToken() override;
+
+  // Helper class that is used to scope batch updates.
+  class ScopedReadingListBatchUpdate
+      : public ReadingListModel::ScopedReadingListBatchUpdate {
+   public:
+    explicit ScopedReadingListBatchUpdate(ReadingListModelImpl* model);
+
+    ~ScopedReadingListBatchUpdate() override;
+
+   private:
+    std::unique_ptr<ReadingListModelStorage::ScopedBatchUpdate> storage_token_;
+
+    DISALLOW_COPY_AND_ASSIGN(ScopedReadingListBatchUpdate);
+  };
+
  protected:
-  void EndBatchUpdates() override;
+  void EnteringBatchUpdates() override;
+  void LeavingBatchUpdates() override;
 
  private:
+  void RemoveEntryByURLImpl(const GURL& url, bool from_sync);
   void SetPersistentHasUnseen(bool has_unseen);
   bool GetPersistentHasUnseen();
+  ReadingListEntry* GetMutableEntryFromURL(const GURL& gurl, bool* read) const;
+  void SortEntries();
+  ReadingListModelStorage* StorageLayer();
 
-  typedef std::vector<ReadingListEntry> ReadingListEntries;
-
-  ReadingListEntries unread_;
-  ReadingListEntries read_;
+  std::unique_ptr<ReadingListEntries> unread_;
+  std::unique_ptr<ReadingListEntries> read_;
   std::unique_ptr<ReadingListModelStorage> storage_layer_;
   PrefService* pref_service_;
   bool has_unseen_;
   bool loaded_;
-
+  base::WeakPtrFactory<ReadingListModelImpl> weak_ptr_factory_;
   DISALLOW_COPY_AND_ASSIGN(ReadingListModelImpl);
 };
 
 #endif  // IOS_CHROME_BROWSER_READING_LIST_READING_LIST_MODEL_MEMORY_H_