diff options
| -rw-r--r-- | chrome/browser/bookmarks/bookmark_model.cc | 5 | ||||
| -rw-r--r-- | chrome/browser/sync/profile_sync_service_unittest.cc | 2 |
2 files changed, 4 insertions, 3 deletions
diff --git a/chrome/browser/bookmarks/bookmark_model.cc b/chrome/browser/bookmarks/bookmark_model.cc index cf9f653..be78ef5 100644 --- a/chrome/browser/bookmarks/bookmark_model.cc +++ b/chrome/browser/bookmarks/bookmark_model.cc @@ -223,13 +223,12 @@ void BookmarkModel::Copy(const BookmarkNode* node, SetDateGroupModified(new_parent, Time::Now()); BookmarkDragData drag_data_(node); std::vector<BookmarkDragData::Element> elements(drag_data_.elements); + // CloneDragData will use BookmarkModel methods to do the job, so we + // don't need to send notifications here. bookmark_utils::CloneDragData(this, elements, new_parent, index); if (store_.get()) store_->ScheduleSave(); - - FOR_EACH_OBSERVER(BookmarkModelObserver, observers_, - BookmarkNodeAdded(this, new_parent, index)); } const SkBitmap& BookmarkModel::GetFavIcon(const BookmarkNode* node) { diff --git a/chrome/browser/sync/profile_sync_service_unittest.cc b/chrome/browser/sync/profile_sync_service_unittest.cc index 515d7d1..7bd90c0 100644 --- a/chrome/browser/sync/profile_sync_service_unittest.cc +++ b/chrome/browser/sync/profile_sync_service_unittest.cc @@ -491,6 +491,8 @@ TEST_F(ProfileSyncServiceTest, BookmarkModelOperations) { ExpectModelMatch(); model_->SetTitle(folder, L"who's nested now?"); ExpectModelMatch(); + model_->Copy(url2, model_->GetBookmarkBarNode(), 0); + ExpectModelMatch(); // Test deletion. // Delete a single item. |