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

chrome/browser/platform_util_chromeos.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/platform_util.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/file_util.h"
 #include "base/utf_string_conversions.h"
@@ skipping 37 matching lines @@
   file_manager_util::ShowFileInFolder(full_path);
 }
 
 void OpenItem(const FilePath& full_path) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   BrowserThread::PostTask(BrowserThread::FILE, FROM_HERE,
       base::Bind(&OpenItemOnFileThread, 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());
     BrowserThread::PostTask(BrowserThread::UI, FROM_HERE,
                             base::Bind(OpenURL, string_url));
   }
 }
 
 }  // namespace platform_util