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

Linux: use new xdg-settings "check" feature to determine whether we are the default browser.
BUG=17093
TEST=in GNOME, let Firefox set itself as the default after Chrome has set itself; Chrome should then detect that it is no longer the default


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=21260

[Evan Stade, 2009-07-22 00:39:37]

http://codereview.chromium.org/155889/diff/1003/6
File chrome/browser/shell_integration_linux.cc (right):

http://codereview.chromium.org/155889/diff/1003/6#newcode81
Line 81: // TODO: Really, being the default browser should be a ternary query:
yes,
this TODO needs your name in it

http://codereview.chromium.org/155889/diff/1003/6#newcode87
Line 87: if (reply.length() > 0 && reply[reply.length() - 1] == '\n')
can you just do strstr for yes

[Mike Mammarella, 2009-07-22 01:34:31]

http://codereview.chromium.org/155889/diff/1003/6
File chrome/browser/shell_integration_linux.cc (right):

http://codereview.chromium.org/155889/diff/1003/6#newcode81
Line 81: // TODO: Really, being the default browser should be a ternary query:
yes,
On 2009/07/22 00:39:37, Evan Stade wrote:
> this TODO needs your name in it

Done.

http://codereview.chromium.org/155889/diff/1003/6#newcode87
Line 87: if (reply.length() > 0 && reply[reply.length() - 1] == '\n')
On 2009/07/22 00:39:37, Evan Stade wrote:
> can you just do strstr for yes

Done.

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

lgtm

http://codereview.chromium.org/155889/diff/10/1008
File chrome/browser/shell_integration_linux.cc (right):

http://codereview.chromium.org/155889/diff/10/1008#newcode86
Line 86: // Allow any reply that starts with "yes"
. at end of comment