Revert of xdg-settings: add secondary check in GetIsDefaultWebClient

Revert of xdg-settings: add secondary check in GetIsDefaultWebClient (patchset #2 id:40001 of https://codereview.chromium.org/2737283002/ )

Reason for revert:
mdm@ says this is the wrong approach

Original issue's description:
> xdg-settings: add secondary check in GetIsDefaultWebClient
> 
> This CL fixes an issue where Chrome would display a banner saying that it
> was not the default browser when it actually was.
> 
> The cause of this was 'xdg-settings check default-web-browser
> google-chrome.desktop' returning 'no' when Chrome might have been the
> default (see further explanation in the CL comments).
> 
> BUG=578888
> R=thestig@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2737283002
> Cr-Commit-Position: refs/heads/master@{#456224}
> Committed: https://chromium.googlesource.com/chromium/src/+/dece487a37276f1043d8092acd01a761ec99c17e

TBR=thestig@chromium.org,mdm@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=578888
Review-Url: https://codereview.chromium.org/2751233002
Cr-Commit-Position: refs/heads/master@{#457272}
Committed: https://chromium.googlesource.com/chromium/src/+/8258b1c5ed8ce2a96dfdba9ce36263a2de2ba768

[Tom (Use chromium acct), 2017-03-15 22:54:24]

The CQ bit was checked by thomasanderson@google.com

[Tom (Use chromium acct), 2017-03-15 22:54:24]

Created Revert of xdg-settings: add secondary check in GetIsDefaultWebClient

[commit-bot: I haz the power, 2017-03-15 22:54:54]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-15 22:54:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-03-15 22:54:56]

No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[Lei Zhang, 2017-03-15 23:11:44]

lgtm

GetIsDefaultWebClient() returns...

[Tom (Use chromium acct), 2017-03-15 23:12:18]

The CQ bit was checked by thomasanderson@google.com

[commit-bot: I haz the power, 2017-03-15 23:12:50]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-03-16 00:04:24]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489619538013420, "parent_rev":
"bf193279edce86f900ae5b222dc93ea427ae1ff0", "commit_rev":
"8258b1c5ed8ce2a96dfdba9ce36263a2de2ba768"}

[commit-bot: I haz the power, 2017-03-16 00:06:19]

Description was changed from

==========
Revert of xdg-settings: add secondary check in GetIsDefaultWebClient (patchset
#2 id:40001 of https://codereview.chromium.org/2737283002/ )

Reason for revert:
mdm@ says this is the wrong approach

Original issue's description:
> xdg-settings: add secondary check in GetIsDefaultWebClient
> 
> This CL fixes an issue where Chrome would display a banner saying that it
> was not the default browser when it actually was.
> 
> The cause of this was 'xdg-settings check default-web-browser
> google-chrome.desktop' returning 'no' when Chrome might have been the
> default (see further explanation in the CL comments).
> 
> BUG=578888
> R=thestig@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2737283002
> Cr-Commit-Position: refs/heads/master@{#456224}
> Committed:
https://chromium.googlesource.com/chromium/src/+/dece487a37276f1043d8092acd01...

TBR=thestig@chromium.org,mdm@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=578888
==========

to

==========
Revert of xdg-settings: add secondary check in GetIsDefaultWebClient (patchset
#2 id:40001 of https://codereview.chromium.org/2737283002/ )

Reason for revert:
mdm@ says this is the wrong approach

Original issue's description:
> xdg-settings: add secondary check in GetIsDefaultWebClient
> 
> This CL fixes an issue where Chrome would display a banner saying that it
> was not the default browser when it actually was.
> 
> The cause of this was 'xdg-settings check default-web-browser
> google-chrome.desktop' returning 'no' when Chrome might have been the
> default (see further explanation in the CL comments).
> 
> BUG=578888
> R=thestig@chromium.org
> 
> Review-Url: https://codereview.chromium.org/2737283002
> Cr-Commit-Position: refs/heads/master@{#456224}
> Committed:
https://chromium.googlesource.com/chromium/src/+/dece487a37276f1043d8092acd01...

TBR=thestig@chromium.org,mdm@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=578888

Review-Url: https://codereview.chromium.org/2751233002
Cr-Commit-Position: refs/heads/master@{#457272}
Committed:
https://chromium.googlesource.com/chromium/src/+/8258b1c5ed8ce2a96dfdba9ce362...
==========

[commit-bot: I haz the power, 2017-03-16 00:06:20]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/8258b1c5ed8ce2a96dfdba9ce362...