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

components/bookmarks/browser/bookmark_utils.h

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..5a65ab4636437a7d578235843f0359661ba5a9ad 100644
--- a/components/bookmarks/browser/bookmark_utils.h
+++ b/components/bookmarks/browser/bookmark_utils.h
@@ -150,6 +150,9 @@ bool IsBookmarkedByUser(BookmarkModel* model, const GURL& url);
 // Returns the node with |id|, or NULL if there is no node with |id|.
 const BookmarkNode* GetBookmarkNodeByID(const BookmarkModel* model, int64 id);
 
+// Checks if clipboard contains valid url
+bool ClipboardContainsValidUrl(base::string16&);

[sky, 2014/08/22 19:40:41]

I strongly urge you to read the style guide. Star with
http://www.chromium.org/developers/coding-style and make sure to visit the c++
specific link.

. Declaration and definition order should list argument names (and they should
be the same).
. Decleration and definition order need to be the same. Additionally declaration
and definition order must be the same. 
. Only allowable references are const. If you need an out param, it should be a
pointer.

Why not make this return a URL?

Having said all this, why both with this in the header? It is only used
internally. It should be in an anonymous namespace in the .cc.

+
 }  // namespace bookmarks
 
 #endif  // COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_UTILS_H_