Add destructor to ReadingListStoreDelegate

components/reading_list/ios/reading_list_store_delegate.h

Index: components/reading_list/ios/reading_list_store_delegate.h
diff --git a/components/reading_list/ios/reading_list_store_delegate.h b/components/reading_list/ios/reading_list_store_delegate.h
index ff4b14b52d1f7778303dacc8355af4f299053fa0..03448c2d66aad0daeababb8c5fe69aee9540da46 100644
--- a/components/reading_list/ios/reading_list_store_delegate.h
+++ b/components/reading_list/ios/reading_list_store_delegate.h
@@ -7,11 +7,16 @@
 
 #include <map>
 
+#include "base/macros.h"
+
 class ReadingListEntry;
 
 // The delegate to handle callbacks from the ReadingListStore.
 class ReadingListStoreDelegate {
  public:
+  ReadingListStoreDelegate() {}
+  virtual ~ReadingListStoreDelegate() {}
+
   using ReadingListEntries = std::map<GURL, ReadingListEntry>;
   // These three mathods handle callbacks from a ReadingListStore.

[sdefresne, 2016/12/11 19:34:51]

nit: can you fix typo here "mathods" should be spelled "methods"

[Olivier, 2016/12/11 19:39:27]

Done.

   virtual void StoreLoaded(std::unique_ptr<ReadingListEntries> entries) = 0;

[sdefresne, 2016/12/11 19:34:51]

nit: this method has no comment (the comment above is common for the three
methods, and the next two methods have comment, just not this one). Can you add
one as a followup?

[Olivier, 2016/12/11 19:39:27]

Done.

@@ -29,6 +34,8 @@ class ReadingListStoreDelegate {
 
   // Called to remove an entry to the model.
   virtual void SyncRemoveEntry(const GURL& url) = 0;
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ReadingListStoreDelegate);
 };
 
 #endif  // COMPONENTS_READING_LIST_IOS_READING_LIST_STORE_DELEGATE_H_