Allow chrome to become the os default handler for arbitrary protocols on mac/win.

chrome/browser/shell_integration_linux.cc

Index: chrome/browser/shell_integration_linux.cc
diff --git a/chrome/browser/shell_integration_linux.cc b/chrome/browser/shell_integration_linux.cc
index e9c7cdda22de7359785f97f4fa2ad1f85f09f50f..135e1aa46b474f32d01889d139932415ea7358cf 100644
--- a/chrome/browser/shell_integration_linux.cc
+++ b/chrome/browser/shell_integration_linux.cc
@@ -254,7 +254,13 @@ bool ShellIntegration::SetAsDefaultBrowser() {
 }
 
 // static
-ShellIntegration::DefaultBrowserState ShellIntegration::IsDefaultBrowser() {
+bool ShellIntegration::SetAsDefaultProtocolClient(const std::string& protocol) {
+  // FIXME: Implement this for Linux - crbug.com/83557

[Mark Mentovai, 2011/05/24 16:28:53]

Here and on line 288, we use TODO, not FIXME.

http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=TODO_C...

[benwells, 2011/05/25 08:07:19]

Done.

+  return false;
+}
+
+// static
+ShellIntegration::DefaultWebClientState ShellIntegration::IsDefaultBrowser() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE));
 
   scoped_ptr<base::Environment> env(base::Environment::Create());
@@ -268,11 +274,19 @@ ShellIntegration::DefaultBrowserState ShellIntegration::IsDefaultBrowser() {
   std::string reply;
   if (!base::GetAppOutput(CommandLine(argv), &reply)) {
     // xdg-settings failed: we can't determine or set the default browser.
-    return UNKNOWN_DEFAULT_BROWSER;
+    return UNKNOWN_DEFAULT_WEB_CLIENT;
   }
 
   // Allow any reply that starts with "yes".
-  return (reply.find("yes") == 0) ? IS_DEFAULT_BROWSER : NOT_DEFAULT_BROWSER;
+  return (reply.find("yes") == 0) ?
+      IS_DEFAULT_WEB_CLIENT : NOT_DEFAULT_WEB_CLIENT;
+}
+
+// static
+ShellIntegration::DefaultWebClientState
+ShellIntegration::IsDefaultProtocolClient(const std::string& protocol) {
+  // FIXME: Implement this for Linux - crbug.com/83557
+  return UNKNOWN_DEFAULT_WEB_CLIENT;
 }
 
 // static