Chromium Code Reviews| 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..8e1ca1a9477af68f93be7831f73b054f3e25bc10 100644 |
| --- a/chrome/browser/shell_integration_linux.cc |
| +++ b/chrome/browser/shell_integration_linux.cc |
| @@ -254,7 +254,12 @@ bool ShellIntegration::SetAsDefaultBrowser() { |
| } |
| // static |
| -ShellIntegration::DefaultBrowserState ShellIntegration::IsDefaultBrowser() { |
| +bool ShellIntegration::SetAsDefaultProtocolClient(const std::string& protocol) { |
| + return false; |
|
Mark Mentovai
2011/05/23 00:23:52
Is there a bug filed to track adding this feature
benwells
2011/05/24 06:10:38
Done.
|
| +} |
| + |
| +// static |
| +ShellIntegration::DefaultClientAppState ShellIntegration::IsDefaultBrowser() { |
| DCHECK(BrowserThread::CurrentlyOn(BrowserThread::FILE)); |
| scoped_ptr<base::Environment> env(base::Environment::Create()); |
| @@ -268,11 +273,18 @@ 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_CLIENT_APP; |
| } |
| // 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_CLIENT_APP : NOT_DEFAULT_CLIENT_APP; |
| +} |
| + |
| +// static |
| +ShellIntegration::DefaultClientAppState |
| +ShellIntegration::IsDefaultProtocolClient(const std::string& protocol) { |
| + return UNKNOWN_DEFAULT_CLIENT_APP; |
| } |
| // static |