Only (and always) put URLs in NSLinkAttributeName.

ui/base/cocoa/controls/hyperlink_text_view.h

Index: ui/base/cocoa/controls/hyperlink_text_view.h
diff --git a/ui/base/cocoa/controls/hyperlink_text_view.h b/ui/base/cocoa/controls/hyperlink_text_view.h
index f04ad40b1823aee1b60fa85e0c5e2b39aec6b907..affe648d206bc012a42a43d04563a24d2867fc65 100644
--- a/ui/base/cocoa/controls/hyperlink_text_view.h
+++ b/ui/base/cocoa/controls/hyperlink_text_view.h
@@ -26,10 +26,9 @@ UI_BASE_EXPORT
           withFont:(NSFont*)font
       messageColor:(NSColor*)messageColor;
 
-// Marks a |range| within the given message as link, associating it with
-// a |name| that is passed to the delegate's textView:clickedOnLink:atIndex:.
+// Marks a |range| within the given message as link.
 - (void)addLinkRange:(NSRange)range
-            withName:(id)name
+             withURL:(NSString*)url

[noyau (Ping after 24h), 2015/09/24 07:54:05]

Driving by: Is there a reason why this is a string? If you expect a URL wouldn't
it better to just type it as a GURL here?

[Avi (use Gerrit), 2015/09/24 14:24:57]

This is a core Cocoa UI class, and everything else is taken in Cocoa types, so
I'm doing that here too. If I were to change to any URL class, it would be
NSURL*.

            linkColor:(NSColor*)linkColor;
 
 // This is NO (by default) if the view rejects first responder status.