Linux: use new xdg-settings "check" feature to determine whether we are the default browser.

chrome/browser/shell_integration_linux.cc

 // Copyright (c) 2006-2009 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/shell_integration.h"
 
 #include <fcntl.h>
 #include <stdlib.h>
 #include <sys/stat.h>
 #include <sys/types.h>
@@ skipping 49 matching lines @@
     close(devnull);
     return false;
   }
   close(devnull);
 
   int success_code;
   base::WaitForExitCode(handle, &success_code);
   return success_code == EXIT_SUCCESS;
 }
 
-static bool GetDefaultBrowser(std::string* result) {
+bool ShellIntegration::IsDefaultBrowser() {
+  std::vector<std::string> argv;
+  argv.push_back("xdg-settings");
+  argv.push_back("check");
+  argv.push_back("default-web-browser");
+  argv.push_back(GetDesktopName());
+
+  std::string reply;
+  if (!base::GetAppOutput(CommandLine(argv), &reply)) {
+    // If xdg-settings fails, we assume that we should pretend we're the default
+    // browser to avoid giving repeated prompts to set ourselves as the default.
+    // TODO(mdm): Really, being the default browser should be a ternary query:
+    // yes, no, and "don't know" so the UI can reflect this more accurately.
+    return true;
+  }
+
+  // Allow any reply that starts with "yes"

[Evan Stade, 2009/07/22 01:36:33]

. at end of comment

+  return reply.find("yes") == 0;
+}
+
+bool ShellIntegration::IsFirefoxDefaultBrowser() {
   std::vector<std::string> argv;
   argv.push_back("xdg-settings");
   argv.push_back("get");
   argv.push_back("default-web-browser");
-  return base::GetAppOutput(CommandLine(argv), result);
-}
 
-bool ShellIntegration::IsDefaultBrowser() {
-  std::string browser;
-  if (!GetDefaultBrowser(&browser)) {
-    // If GetDefaultBrowser() fails, we assume xdg-settings does not work for
-    // some reason, and that we should pretend we're the default browser to
-    // avoid giving repeated prompts to set ourselves as the default browser.
-    // TODO: Really, being the default browser should be a ternary query: yes,
-    // no, and "don't know" so that the UI can reflect this more accurately.
-    return true;
-  }
-  // Allow for an optional newline at the end.
-  if (browser.length() > 0 && browser[browser.length() - 1] == '\n')
-    browser.resize(browser.length() - 1);
-  return !browser.compare(GetDesktopName());
-}
-
-bool ShellIntegration::IsFirefoxDefaultBrowser() {
   std::string browser;
   // We don't care about the return value here.
-  GetDefaultBrowser(&browser);
+  base::GetAppOutput(CommandLine(argv), &browser);
   return browser.find("irefox") != std::string::npos;
 }