Supporting undoing bookmark deletion without creating new ID.

components/bookmarks/browser/bookmark_undo_delegate.h

+// Copyright 2015 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_BOOKMARKS_BROWSER_BOOKMARK_UNDO_DELEGATE_H_
+#define COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_UNDO_DELEGATE_H_
+
+#include "base/memory/scoped_ptr.h"
+
+namespace bookmarks {
+
+class BookmarkModel;
+class BookmarkNode;
+class BookmarkUndoProvider;
+
+class BookmarkUndoDelegate {

[sky, 2015/10/08 15:20:01]

Add a description.

[jianli, 2015/10/08 23:37:40]

Done.

+ public:
+  // Called when |node| was deleted from |parent| at position |index|. The

[sky, 2015/10/08 15:20:00]

deleted->rmeoted

[jianli, 2015/10/08 23:37:40]

Done.

+  // delegate takes the ownership of the deleted node.

[sky, 2015/10/08 15:20:01]

The second sentence is no longer needed as node is a scoped_ptr.

[jianli, 2015/10/08 23:37:40]

Done.

+  virtual void OnBookmarkNodeDeleted(BookmarkModel* model,

[sky, 2015/10/08 15:20:00]

OnBookmarkNodeRemoved.

[jianli, 2015/10/08 23:37:40]

Done.

+                                     BookmarkUndoProvider* provider,

[sky, 2015/10/08 15:20:00]

Rather than supply the provider on event call how a SetProvider function?

[jianli, 2015/10/08 23:37:40]

Done.

+                                     const BookmarkNode* parent,
+                                     int index,
+                                     scoped_ptr<BookmarkNode> node) = 0;
+
+ protected:
+  virtual ~BookmarkUndoDelegate() {}
+};
+
+
+}  // namespace bookmarks
+
+#endif  // COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_UNDO_DELEGATE_H_