Show paste option on right click of bookmark bar if some URL is copied from outside browser or from…

components/bookmarks/browser/bookmark_node_data.cc

Index: components/bookmarks/browser/bookmark_node_data.cc
diff --git a/components/bookmarks/browser/bookmark_node_data.cc b/components/bookmarks/browser/bookmark_node_data.cc
index ad2ad42476dd5defdad41c779b04df457958c6f6..92bbf5c8a8e238414762a6f81acb5a46c9660aca 100644
--- a/components/bookmarks/browser/bookmark_node_data.cc
+++ b/components/bookmarks/browser/bookmark_node_data.cc
@@ -121,6 +121,23 @@ bool BookmarkNodeData::ClipboardContainsBookmarks() {
       ui::Clipboard::GetFormatType(kClipboardFormatString),
       ui::CLIPBOARD_TYPE_COPY_PASTE);
 }
+
+bool BookmarkNodeData::ClipboardContainsURLs() {

[sky, 2014/08/05 16:53:49]

These functions don't need bookmarkNodeData at all. They should live somewhere
else so that this code remains focused.

[tfarina, 2014/08/05 16:56:48]

Scott, would you be fine with bookmark_clipboard.*. I have always wanted to move
some clipboard functions out of bookmark_utils.* into bookmark_clipboard.*
anyway.

Maybe now, it is time to start that move?

+  return ui::Clipboard::GetForCurrentThread()->IsFormatAvailable(
+      ui::Clipboard::GetUrlWFormatType(), ui::CLIPBOARD_TYPE_COPY_PASTE);
+}
+
+bool BookmarkNodeData::WriteBookmarkToClipboardFromURL() {
+  BookmarkNodeData data;
+  base::string16 url_text;
+  ui::Clipboard::GetForCurrentThread()->ReadText(ui::CLIPBOARD_TYPE_COPY_PASTE,
+                                                 &url_text);
+  GURL url(url_text);
+  if (!data.ReadFromTuple(url, url_text))
+    return false;
+  data.WriteToClipboard(ui::CLIPBOARD_TYPE_COPY_PASTE);
+  return true;
+}
 #endif
 
 bool BookmarkNodeData::ReadFromVector(