Title: Same Bookmark url is getting pasted on the Bookmarkbar with same title.

components/bookmarks/browser/bookmark_utils_unittest.cc

Index: components/bookmarks/browser/bookmark_utils_unittest.cc
diff --git a/components/bookmarks/browser/bookmark_utils_unittest.cc b/components/bookmarks/browser/bookmark_utils_unittest.cc
index 7a94080a4f94b666f8be64136cfdcb7598a2aa69..745fd9f9148e689d7f00e4f19988fe2f9467e1d5 100644
--- a/components/bookmarks/browser/bookmark_utils_unittest.cc
+++ b/components/bookmarks/browser/bookmark_utils_unittest.cc
@@ -314,6 +314,40 @@ TEST_F(BookmarkUtilsTest, CopyPaste) {
   EXPECT_FALSE(CanPasteFromClipboard(model.get(), model->bookmark_bar_node()));
 }
 
+TEST_F(BookmarkUtilsTest, PasteSameBookmarkedURL) {

[sky, 2014/10/10 17:32:45]

Either add a description of what this test is verifying, or make the title
reflect that. Right now the name is overly generic and doesn't indicate what
this is asserting.

[Deepak, 2014/10/11 06:32:38]

Done.

+  TestBookmarkClient client;
+  scoped_ptr<BookmarkModel> model(client.CreateModel());
+  const base::string16 url_text = ASCIIToUTF16("http://www.google.com/");
+  const base::string16 title_text = ASCIIToUTF16("foobar");
+  const base::string16 new_title_text = ASCIIToUTF16("foobar (1)");
+  const BookmarkNode* bookmark_bar_node = model->bookmark_bar_node();
+
+  const BookmarkNode* node =
+      model->AddURL(bookmark_bar_node, 0, title_text, GURL(url_text));
+
+  EXPECT_EQ(url_text,
+            ASCIIToUTF16(bookmark_bar_node->GetChild(0)->url().spec()));
+  EXPECT_EQ(title_text, bookmark_bar_node->GetChild(0)->GetTitle());
+
+  // Copy a node to the clipboard.
+  std::vector<const BookmarkNode*> nodes;
+  nodes.push_back(node);
+  CopyToClipboard(model.get(), nodes, false);
+
+  // Now we should be able to paste from the clipboard.
+  EXPECT_TRUE(CanPasteFromClipboard(model.get(), bookmark_bar_node));
+
+  PasteFromClipboard(model.get(), bookmark_bar_node, 1);
+  ASSERT_EQ(2, bookmark_bar_node->child_count());
+
+  // Url for added node should be same as url_text.
+  EXPECT_EQ(url_text,
+            ASCIIToUTF16(bookmark_bar_node->GetChild(1)->url().spec()));
+  // Title for added node should be numeric subscript suffix with copied node
+  // title.
+  EXPECT_EQ(new_title_text, bookmark_bar_node->GetChild(1)->GetTitle());
+}
+
 TEST_F(BookmarkUtilsTest, CopyPasteMetaInfo) {
   TestBookmarkClient client;
   scoped_ptr<BookmarkModel> model(client.CreateModel());