[NetworkChangeNotifier] Run Windows connection type computation on safe thread

[NetworkChangeNotifier] Run Windows connection type computation on safe thread

*Merge to M60*

1. Move RecomputeCurrentConnectionType to anonymous namespace
2. Call RecomputeCurrentConnectionType via callback so tests can override
3. Call RecomputeCurrentConnectionType on dns thread
4. Tests inject their own dns task runner

BUG=721461, 725487
Review-Url: https://codereview.chromium.org/2893943002
Cr-Commit-Position: refs/heads/master@{#475082}
(cherry picked from commit 6b0ea35f8c93c6eb229ba09624d566a339b3073b)

TBR=pauljensen

Review-Url: https://codereview.chromium.org/2914833002 .
Cr-Commit-Position: refs/branch-heads/3112@{#59}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed: https://chromium.googlesource.com/chromium/src/+/6da688ab3117d09406f40fd3395c6874d0d39039

[jkarlin, 2017-05-31 15:15:42]

Description was changed from

==========
[NetworkChangeNotifier] Run Windows connection type computation on safe thread

1. Move RecomputeCurrentConnectionType to anonymous namespace
2. Call RecomputeCurrentConnectionType via callback so tests can override
3. Call RecomputeCurrentConnectionType on dns thread
4. Tests inject their own dns task runner

BUG=721461,725487

Review-Url: https://codereview.chromium.org/2893943002
Cr-Commit-Position: refs/heads/master@{#475082}
(cherry picked from commit 6b0ea35f8c93c6eb229ba09624d566a339b3073b)
==========

to

==========
[NetworkChangeNotifier] Run Windows connection type computation on safe thread

*Merge to M60*

1. Move RecomputeCurrentConnectionType to anonymous namespace
2. Call RecomputeCurrentConnectionType via callback so tests can override
3. Call RecomputeCurrentConnectionType on dns thread
4. Tests inject their own dns task runner

BUG=721461,725487

Review-Url: https://codereview.chromium.org/2893943002
Cr-Commit-Position: refs/heads/master@{#475082}
(cherry picked from commit 6b0ea35f8c93c6eb229ba09624d566a339b3073b)
==========

[jkarlin, 2017-05-31 15:17:34]

Description was changed from

==========
[NetworkChangeNotifier] Run Windows connection type computation on safe thread

*Merge to M60*

1. Move RecomputeCurrentConnectionType to anonymous namespace
2. Call RecomputeCurrentConnectionType via callback so tests can override
3. Call RecomputeCurrentConnectionType on dns thread
4. Tests inject their own dns task runner

BUG=721461,725487

Review-Url: https://codereview.chromium.org/2893943002
Cr-Commit-Position: refs/heads/master@{#475082}
(cherry picked from commit 6b0ea35f8c93c6eb229ba09624d566a339b3073b)
==========

to

==========
[NetworkChangeNotifier] Run Windows connection type computation on safe thread

*Merge to M60*

1. Move RecomputeCurrentConnectionType to anonymous namespace
2. Call RecomputeCurrentConnectionType via callback so tests can override
3. Call RecomputeCurrentConnectionType on dns thread
4. Tests inject their own dns task runner

BUG=721461,725487

Review-Url: https://codereview.chromium.org/2893943002
Cr-Commit-Position: refs/heads/master@{#475082}
(cherry picked from commit 6b0ea35f8c93c6eb229ba09624d566a339b3073b)

TBR=pauljensen
==========

[jkarlin, 2017-05-31 15:17:49]

jkarlin@chromium.org changed reviewers:
+ pauljensen@chromium.org

[jkarlin, 2017-05-31 17:29:28]

Description was changed from

==========
[NetworkChangeNotifier] Run Windows connection type computation on safe thread

*Merge to M60*

1. Move RecomputeCurrentConnectionType to anonymous namespace
2. Call RecomputeCurrentConnectionType via callback so tests can override
3. Call RecomputeCurrentConnectionType on dns thread
4. Tests inject their own dns task runner

BUG=721461,725487

Review-Url: https://codereview.chromium.org/2893943002
Cr-Commit-Position: refs/heads/master@{#475082}
(cherry picked from commit 6b0ea35f8c93c6eb229ba09624d566a339b3073b)

TBR=pauljensen
==========

to

==========
[NetworkChangeNotifier] Run Windows connection type computation on safe thread

*Merge to M60*

1. Move RecomputeCurrentConnectionType to anonymous namespace
2. Call RecomputeCurrentConnectionType via callback so tests can override
3. Call RecomputeCurrentConnectionType on dns thread
4. Tests inject their own dns task runner

BUG=721461,725487

Review-Url: https://codereview.chromium.org/2893943002
Cr-Commit-Position: refs/heads/master@{#475082}
(cherry picked from commit 6b0ea35f8c93c6eb229ba09624d566a339b3073b)

TBR=pauljensen

Review-Url: https://codereview.chromium.org/2914833002 .
Cr-Commit-Position: refs/branch-heads/3112@{#59}
Cr-Branched-From:
b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}
Committed:
https://chromium.googlesource.com/chromium/src/+/6da688ab3117d09406f40fd3395c...
==========

[jkarlin, 2017-05-31 17:29:29]

Committed patchset #1 (id:1) manually as
6da688ab3117d09406f40fd3395c6874d0d39039.