Fix inheritance in Reading List

components/reading_list/ios/reading_list_model_storage.h

Index: components/reading_list/ios/reading_list_model_storage.h
diff --git a/components/reading_list/ios/reading_list_model_storage.h b/components/reading_list/ios/reading_list_model_storage.h
index 422abf04367c90b1f95949f37837e2595d8cff7d..cca1b78cf039c757fc02e6350aa07af352e5a2dd 100644
--- a/components/reading_list/ios/reading_list_model_storage.h
+++ b/components/reading_list/ios/reading_list_model_storage.h
@@ -9,6 +9,8 @@
 
 #include "base/macros.h"
 #include "components/reading_list/ios/reading_list_entry.h"
+#include "components/sync/base/model_type.h"
+#include "components/sync/model/model_type_sync_bridge.h"
 
 class ReadingListModel;
 class ReadingListStoreDelegate;
@@ -19,21 +21,20 @@ class ModelTypeSyncBridge;
 
 // Interface for a persistence layer for reading list.
 // All interface methods have to be called on main thread.
-class ReadingListModelStorage {
+class ReadingListModelStorage : public syncer::ModelTypeSyncBridge {
  public:
   class ScopedBatchUpdate;
 
-  ReadingListModelStorage() {}
-  virtual ~ReadingListModelStorage() {}
+  ReadingListModelStorage(
+      const ChangeProcessorFactory& change_processor_factory,
+      syncer::ModelType type);
+  ~ReadingListModelStorage() override{};

[sdefresne, 2016/12/12 09:33:24]

required: no semi-colon after method implementation

optional: since you have a .cc for the constructor implementation, I would
personally put the destructor implementation in the .cc file too.

[Olivier, 2016/12/12 10:20:17]

Done.

 
   // Sets the model the Storage is backing.
   // This will trigger store initalization and load persistent entries.
   virtual void SetReadingListModel(ReadingListModel* model,
                                    ReadingListStoreDelegate* delegate) = 0;
 
-  // Returns the class responsible for handling sync messages.
-  virtual syncer::ModelTypeSyncBridge* GetModelTypeSyncBridge() = 0;
-
   // Starts a transaction. All Save/Remove entry will be delayed until the
   // transaction is commited.
   // Multiple transaction can be started at the same time. Commit will happen