Chromium Code Reviews| Index: components/undo/bookmark_undo_service.cc |
| diff --git a/components/undo/bookmark_undo_service.cc b/components/undo/bookmark_undo_service.cc |
| index cdac524cc6e0ea0d01cd88aa66da5f856c95c37f..62a6f8f4e8880af51f00b0741914062b147e2e5a 100644 |
| --- a/components/undo/bookmark_undo_service.cc |
| +++ b/components/undo/bookmark_undo_service.cc |
| @@ -6,6 +6,8 @@ |
| #include <stddef.h> |
| #include <stdint.h> |
| + |
| +#include <memory> |
|
danakj
2016/04/25 18:32:58
dont need then
dcheng
2016/04/25 19:55:06
See previous.
|
| #include <utility> |
| #include "base/macros.h" |
| @@ -100,7 +102,7 @@ class BookmarkRemoveOperation : public BookmarkUndoOperation { |
| BookmarkUndoProvider* undo_provider, |
| const BookmarkNode* parent, |
| int index, |
| - scoped_ptr<BookmarkNode> node); |
| + std::unique_ptr<BookmarkNode> node); |
| ~BookmarkRemoveOperation() override; |
| // UndoOperation: |
| @@ -112,7 +114,7 @@ class BookmarkRemoveOperation : public BookmarkUndoOperation { |
| BookmarkUndoProvider* undo_provider_; |
| const int64_t parent_node_id_; |
| const int index_; |
| - scoped_ptr<BookmarkNode> node_; |
| + std::unique_ptr<BookmarkNode> node_; |
| DISALLOW_COPY_AND_ASSIGN(BookmarkRemoveOperation); |
| }; |
| @@ -122,7 +124,7 @@ BookmarkRemoveOperation::BookmarkRemoveOperation( |
| BookmarkUndoProvider* undo_provider, |
| const BookmarkNode* parent, |
| int index, |
| - scoped_ptr<BookmarkNode> node) |
| + std::unique_ptr<BookmarkNode> node) |
| : BookmarkUndoOperation(model), |
| undo_provider_(undo_provider), |
| parent_node_id_(parent->id()), |
| @@ -364,7 +366,7 @@ void BookmarkUndoService::BookmarkNodeMoved(BookmarkModel* model, |
| int old_index, |
| const BookmarkNode* new_parent, |
| int new_index) { |
| - scoped_ptr<UndoOperation> op(new BookmarkMoveOperation( |
| + std::unique_ptr<UndoOperation> op(new BookmarkMoveOperation( |
| model, old_parent, old_index, new_parent, new_index)); |
| undo_manager()->AddUndoOperation(std::move(op)); |
| } |
| @@ -372,20 +374,20 @@ void BookmarkUndoService::BookmarkNodeMoved(BookmarkModel* model, |
| void BookmarkUndoService::BookmarkNodeAdded(BookmarkModel* model, |
| const BookmarkNode* parent, |
| int index) { |
| - scoped_ptr<UndoOperation> op( |
| + std::unique_ptr<UndoOperation> op( |
| new BookmarkAddOperation(model, parent, index)); |
| undo_manager()->AddUndoOperation(std::move(op)); |
| } |
| void BookmarkUndoService::OnWillChangeBookmarkNode(BookmarkModel* model, |
| const BookmarkNode* node) { |
| - scoped_ptr<UndoOperation> op(new BookmarkEditOperation(model, node)); |
| + std::unique_ptr<UndoOperation> op(new BookmarkEditOperation(model, node)); |
| undo_manager()->AddUndoOperation(std::move(op)); |
| } |
| void BookmarkUndoService::OnWillReorderBookmarkNode(BookmarkModel* model, |
| const BookmarkNode* node) { |
| - scoped_ptr<UndoOperation> op(new BookmarkReorderOperation(model, node)); |
| + std::unique_ptr<UndoOperation> op(new BookmarkReorderOperation(model, node)); |
| undo_manager()->AddUndoOperation(std::move(op)); |
| } |
| @@ -402,12 +404,13 @@ void BookmarkUndoService::SetUndoProvider(BookmarkUndoProvider* undo_provider) { |
| undo_provider_ = undo_provider; |
| } |
| -void BookmarkUndoService::OnBookmarkNodeRemoved(BookmarkModel* model, |
| - const BookmarkNode* parent, |
| - int index, |
| - scoped_ptr<BookmarkNode> node) { |
| +void BookmarkUndoService::OnBookmarkNodeRemoved( |
| + BookmarkModel* model, |
| + const BookmarkNode* parent, |
| + int index, |
| + std::unique_ptr<BookmarkNode> node) { |
| DCHECK(undo_provider_); |
| - scoped_ptr<UndoOperation> op(new BookmarkRemoveOperation( |
| + std::unique_ptr<UndoOperation> op(new BookmarkRemoveOperation( |
| model, undo_provider_, parent, index, std::move(node))); |
| undo_manager()->AddUndoOperation(std::move(op)); |
| } |