Convert FILE thread usage in system_network_api.cc to TaskScheduler.

Convert FILE thread usage in system_network_api.cc to TaskScheduler.

BUG=689520
Review-Url: https://codereview.chromium.org/2975153002
Cr-Commit-Position: refs/heads/master@{#486625}
Committed: https://chromium.googlesource.com/chromium/src/+/b49416e00e80161deee89efceaa0e8d6d9c602f8

[tapted, 2017-07-13 07:05:22]

The CQ bit was checked by tapted@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-07-13 07:05:40]

Dry run: CQ is trying da patch.

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

[tapted, 2017-07-13 07:22:37]

tapted@chromium.org changed reviewers:
+ benwells@chromium.org

[tapted, 2017-07-13 07:22:38]

Hiya ben, please take a look.

(tested with

extensions_unittests --gtest_filter=SystemNetworkApiUnitTest.*
extensions_browsertests --gtest_filter=SystemNetworkApiTest.*

[tapted, 2017-07-13 07:25:33]

The CQ bit was checked by tapted@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-07-13 07:25:57]

Dry run: CQ is trying da patch.

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

[benwells, 2017-07-13 08:24:30]

lgtm withanit

https://codereview.chromium.org/2975153002/diff/20001/extensions/browser/api/...
File extensions/browser/api/system_network/system_network_api.cc (right):

https://codereview.chromium.org/2975153002/diff/20001/extensions/browser/api/...
extensions/browser/api/system_network/system_network_api.cc:12:
std::unique_ptr<net::NetworkInterfaceList> GetListOnFileThread() {
Nit: rename to something that doesn't have "File" in it. Maybe
GetListOnBlockingTaskRunner?

[commit-bot: I haz the power, 2017-07-13 08:36:50]

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

[commit-bot: I haz the power, 2017-07-13 08:36:51]

Dry run: This issue passed the CQ dry run.

[tapted, 2017-07-13 23:56:09]

The CQ bit was checked by tapted@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-07-13 23:56:27]

Dry run: CQ is trying da patch.

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

[tapted, 2017-07-13 23:57:36]

The CQ bit was unchecked by tapted@chromium.org

[tapted, 2017-07-13 23:57:49]

Thanks ben!

https://codereview.chromium.org/2975153002/diff/20001/extensions/browser/api/...
File extensions/browser/api/system_network/system_network_api.cc (right):

https://codereview.chromium.org/2975153002/diff/20001/extensions/browser/api/...
extensions/browser/api/system_network/system_network_api.cc:12:
std::unique_ptr<net::NetworkInterfaceList> GetListOnFileThread() {
On 2017/07/13 08:24:30, benwells wrote:
> Nit: rename to something that doesn't have "File" in it. Maybe
> GetListOnBlockingTaskRunner?

Done.

[tapted, 2017-07-13 23:58:24]

The CQ bit was checked by tapted@chromium.org

[tapted, 2017-07-13 23:58:25]

The patchset sent to the CQ was uploaded after l-g-t-m from
benwells@chromium.org
Link to the patchset: https://codereview.chromium.org/2975153002/#ps40001
(title: "GetListOnBlockingTaskRunner")

[commit-bot: I haz the power, 2017-07-13 23:58:51]

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-07-14 01:58:14]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1499990304951570,
"parent_rev": "c7d73a9906e9e520adf11ebebc178ab0d69147d4", "commit_rev":
"b49416e00e80161deee89efceaa0e8d6d9c602f8"}

[commit-bot: I haz the power, 2017-07-14 01:58:35]

Description was changed from

==========
Convert FILE thread usage in system_network_api.cc to TaskScheduler.

BUG=689520
==========

to

==========
Convert FILE thread usage in system_network_api.cc to TaskScheduler.

BUG=689520

Review-Url: https://codereview.chromium.org/2975153002
Cr-Commit-Position: refs/heads/master@{#486625}
Committed:
https://chromium.googlesource.com/chromium/src/+/b49416e00e80161deee89efceaa0...
==========

[commit-bot: I haz the power, 2017-07-14 01:58:36]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/b49416e00e80161deee89efceaa0...