Installer: Do not write "URL Protocol" to the ChromeHTML registry entry.

chrome/installer/util/shell_util.cc

Index: chrome/installer/util/shell_util.cc
diff --git a/chrome/installer/util/shell_util.cc b/chrome/installer/util/shell_util.cc
index 8a8c00227781a077a558d38a2695de780facb412..1e67163d9173198f8f002a714d59d160dfbf932c 100644
--- a/chrome/installer/util/shell_util.cc
+++ b/chrome/installer/util/shell_util.cc
@@ -297,8 +297,6 @@ class RegistryEntry {
     entries->push_back(new RegistryEntry(
         chrome_html_prog_id, dist->GetBrowserProgIdDesc()));
     entries->push_back(new RegistryEntry(
-        chrome_html_prog_id, ShellUtil::kRegUrlProtocol, base::string16()));

[gab, 2014/09/12 11:43:15]

From
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/installer/u...
this appears to be intended to hook functionality in IE/Explorer. WDYT?

[grt (UTC plus 2), 2014/09/12 13:59:41]

If http://msdn.microsoft.com/library/ie/aa767914.aspx is to be believed, "URL
Protocol" should be added to the HKCR\<proto> key where <proto> names an URL
scheme (e.g., "mailto"). I can't think of any reason to put this value on any
keys except those for URL schemes.

It looks like ShellUtil::RegisterChromeForProtocol doesn't do this registration.
I wonder if Chrome is actually being launched from IE when clicking on links
it's the default handler for.

[Matt Giuca, 2014/09/12 23:49:24]

I'll do some testing in IE on Monday. (I didn't realise this was for IE's
benefit, but for other apps like Start -> Run that say "launch this URL".) It
doesn't make sense for a web browser to respect this setting when the user
clicks on a HTTP link; I would imagine IE only honours this when clicking on a
link it can't handle (like a "steam://" URL).

[grt (UTC plus 2), 2014/09/13 03:27:38]

Exactly. I wonder what would happen if Chrome is registered as the default
protocol handler for, say, webcal://. Will clicking those links in IE launch
Chrome? My guess is that the answer is no by accident. Would you find a way to
trigger navigator.registerProtocolHandler so that Chrome registers itself as the
handler for a protocol, then follow a link for that protocol in IE to see what
happens? This is a great find. I think you may have come across a legitimate bug
in rPH.

[Matt Giuca, 2014/09/15 00:52:01]

No, it works perfectly:

1. Go here: http://www.lookout.net/test/handler/
2. Next to "Run ad hoc test", change "mailto" to "web+foo" and click the button.
3. In IE, type into the URL bar "web+foo:bar".

It will correctly open Chrome and navigate to
http://www.lookout.net/foo=web%2Bfoo%3Abar.

ShellUtil::MakeChromeDefaultProtocolClient is called from
navigator.registerProtocolHandler.

[grt (UTC plus 2), 2014/09/15 13:55:18]

Awesome, thanks for confirming.

-    entries->push_back(new RegistryEntry(
         chrome_html_prog_id + ShellUtil::kRegDefaultIcon, icon_path));
     entries->push_back(new RegistryEntry(
         chrome_html_prog_id + ShellUtil::kRegShellOpen, open_cmd));