Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1502)

Unified Diff: components/bookmarks/browser/bookmark_utils.h

Issue 446003002: Title: Same Bookmark url is getting pasted on the Bookmarkbar with same title. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Changing vector to hash_set as suggested. Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/bookmarks/browser/bookmark_utils.h
diff --git a/components/bookmarks/browser/bookmark_utils.h b/components/bookmarks/browser/bookmark_utils.h
index 9fcbfb57192394e11202c0143c471c0819365662..94e016279ff9332e5d353e91899f0a5abca5b251 100644
--- a/components/bookmarks/browser/bookmark_utils.h
+++ b/components/bookmarks/browser/bookmark_utils.h
@@ -54,6 +54,12 @@ void CopyToClipboard(BookmarkModel* model,
const std::vector<const BookmarkNode*>& nodes,
bool remove_nodes);
+// This helper function will give new title for the bookmark when same
+// bookmark already exist under parent node.
+void UpdateCopiedBookmarkTitle(BookmarkModel* model,
sky 2014/09/30 15:45:16 Why is this in the header? AFAICT it's only needed
Deepak 2014/10/01 05:13:10 Done.
+ const BookmarkNode* parent,
+ BookmarkNodeData* bookmark_data);
+
// Pastes from the clipboard. The new nodes are added to |parent|, unless
// |parent| is null in which case this does nothing. The nodes are inserted
// at |index|. If |index| is -1 the nodes are added to the end.
« no previous file with comments | « no previous file | components/bookmarks/browser/bookmark_utils.cc » ('j') | components/bookmarks/browser/bookmark_utils.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698