Preventing our default handlers for ChromeOS to show up or confuse the user

chrome/browser/platform_util_chromeos.cc

Index: chrome/browser/platform_util_chromeos.cc
diff --git a/chrome/browser/platform_util_chromeos.cc b/chrome/browser/platform_util_chromeos.cc
index eaebafd1c823924b69dcd06f5403c3c1aa22422a..f76f47d31a16dd44ad775dcc5258278876906f45 100644
--- a/chrome/browser/platform_util_chromeos.cc
+++ b/chrome/browser/platform_util_chromeos.cc
@@ -55,6 +55,10 @@ void OpenItem(const FilePath& full_path) {
 }
 
 void OpenExternal(const GURL& url) {
+  // TODO(skuhne): This code seems obsolete since we have default handlers on
+  // ChromeOS which should handle this. The simple "mailto" request default
+  // case was handled properly, but it is unclear if there aren't some hidden
+  // cases left.

[DaveMoore, 2012/04/26 14:45:50]

I would rather see us remove this or replace the comment with why it's necessary
to leave it instead of a TODO.

[Mr4D (OOO till 08-26), 2012/04/26 16:34:37]

Okay, I have changed the comment after reviewing the code paths thoroughly.
There are some cases in which we might break things when removing this.

   if (url.SchemeIs("mailto")) {
     std::string string_url = kGmailComposeUrl;
     string_url.append(url.spec());