Reading List create protobuf store

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 "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"
 
 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 {
  public:
   // Initialize a ReadingListModelImpl to load and save data in
   // |persistence_layer|.
-  ReadingListModelImpl(std::unique_ptr<ReadingListModelStorage> storage_layer);
+  ReadingListModelImpl(std::unique_ptr<ReadingListModelStorage> storage_layer,
+                       PrefService* pref_service);
 
   // Initialize a ReadingListModelImpl without persistence. Data will not be
   // persistent across sessions.
   ReadingListModelImpl();
 
   ~ReadingListModelImpl() override;
   void Shutdown() override;
 
   bool loaded() const override;
 
@@ skipping 18 matching lines @@
 
   void MarkReadByURL(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;
 
+  virtual void ModelLoaded(std::unique_ptr<ReadingListEntries> unread,

[sdefresne, 2016/10/07 09:44:30]

Why use std::unique_ptr<std::vector<ReadingListEntries>>? std::vector<> is
movable, so if you really want to move the value, pass them by value and tell
client code to std::move() them.

[jif-google, 2016/10/07 09:51:34]

1/ The fact that ReadingListEntries is movable is an implementation detail.
2/ Passing by value makes it easy to accidentally copy the object.

+                           std::unique_ptr<ReadingListEntries> read);
+
  protected:
-  void EndBatchUpdates() override;
+  void EnteringBatchUpdates() override;
+  void LeavingBatchUpdates() override;
 
  private:
-  typedef std::vector<ReadingListEntry> ReadingListEntries;
+  void SavePersistentHasUnseen(bool has_unseen);
+  bool LoadPersistentHasUnseen();
 
-  ReadingListEntries unread_;
-  ReadingListEntries read_;
-  std::unique_ptr<ReadingListModelStorage> storageLayer_;
-  bool hasUnseen_;
+  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_