Ignore title parameter for navigator.registerProtocolHandler.

chrome/browser/custom_handlers/register_protocol_handler_infobar_delegate.cc

Index: chrome/browser/custom_handlers/register_protocol_handler_infobar_delegate.cc
diff --git a/chrome/browser/custom_handlers/register_protocol_handler_infobar_delegate.cc b/chrome/browser/custom_handlers/register_protocol_handler_infobar_delegate.cc
index e3027ccf4599af725458edb538e4d1e1f11b06f4..ca2b7fed43db49e7cd89439ba36b1d218310b1dd 100644
--- a/chrome/browser/custom_handlers/register_protocol_handler_infobar_delegate.cc
+++ b/chrome/browser/custom_handlers/register_protocol_handler_infobar_delegate.cc
@@ -72,11 +72,13 @@ RegisterProtocolHandlerInfoBarDelegate*
 base::string16 RegisterProtocolHandlerInfoBarDelegate::GetMessageText() const {
   ProtocolHandler old_handler = registry_->GetHandlerFor(handler_.protocol());
   return old_handler.IsEmpty() ?
-      l10n_util::GetStringFUTF16(IDS_REGISTER_PROTOCOL_HANDLER_CONFIRM,
-          handler_.title(), base::UTF8ToUTF16(handler_.url().host()),

[jochen (gone - plz use gerrit), 2014/05/06 18:38:30]

is title() used anywhere else? Otherwise, you should rip it out entirely

[meacer, 2014/05/06 18:46:50]

I was actually looking at it now: The content settings page uses the title
instead of the url under the "Site" column. mailto and webcal are initiated as
"Google.com Mail" and "Google Calendar" on Chrome OS. Of course nothing prevents
another site from using the same names.

+      l10n_util::GetStringFUTF16(
+          IDS_REGISTER_PROTOCOL_HANDLER_CONFIRM,
+          base::UTF8ToUTF16(handler_.url().host()),
           GetProtocolName(handler_)) :
-      l10n_util::GetStringFUTF16(IDS_REGISTER_PROTOCOL_HANDLER_CONFIRM_REPLACE,
-          handler_.title(), base::UTF8ToUTF16(handler_.url().host()),
+      l10n_util::GetStringFUTF16(
+          IDS_REGISTER_PROTOCOL_HANDLER_CONFIRM_REPLACE,
+          base::UTF8ToUTF16(handler_.url().host()),
           GetProtocolName(handler_), old_handler.title());
 }