Add destructor to ReadingListStoreDelegate

components/reading_list/ios/reading_list_store_delegate.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 COMPONENTS_READING_LIST_IOS_READING_LIST_STORE_DELEGATE_H_
 #define COMPONENTS_READING_LIST_IOS_READING_LIST_STORE_DELEGATE_H_
 
 #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.

   // Handle sync events.
   // Called to add a new entry to the model.
   // |entry| must not already exist in the model.
   virtual void SyncAddEntry(std::unique_ptr<ReadingListEntry> entry) = 0;
 
   // Called to merge a sync entry with a local entry in the model.
   // A local entry with the same URL must exist in the local store and have an
   // older UpdateTime.
   // Return a pointer to the merged entry.
   virtual ReadingListEntry* SyncMergeEntry(
       std::unique_ptr<ReadingListEntry> entry) = 0;
 
   // 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_