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..bf8eb84b24c5f1a2c75746b7a322aa63d95dc9c3 100644
--- a/chrome/browser/platform_util_chromeos.cc
+++ b/chrome/browser/platform_util_chromeos.cc
@@ -55,12 +55,6 @@ void OpenItem(const FilePath& full_path) {
 }
 
 void OpenExternal(const GURL& url) {
-  if (url.SchemeIs("mailto")) {

[benwells, 2012/04/26 00:34:42]

I know I asked for this change, but ... we discussed it on chat and I thought
had decided to add a TODO.

There are a few callers to OpenExternal (e.g. Browser::EmailPageLocation). Have
you check this won't break any of them?

[Mr4D (OOO till 08-26), 2012/04/26 13:46:35]

Well - I have tried all cases I could come up with and they worked as expected.
However - you are right there might be some other use cases I don't know. So I
was adding it back in and added a comment.

-    std::string string_url = kGmailComposeUrl;
-    string_url.append(url.spec());
-    BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
-                            base::Bind(OpenURL, string_url));
-  }
 }
 
 }  // namespace platform_util