Use TaskScheduler instead of PostTask in P2PSocketDispatcherHost.

Use TaskScheduler instead of PostTask in P2PSocketDispatcherHost.

Using USER_VISIBLE priority, since updating the network list may be
needed to establish a WebRTC connection initiated by the user.

BUG=chromium:667892
Review-Url: https://codereview.chromium.org/2963733002
Cr-Commit-Position: refs/heads/master@{#483342}
Committed: https://chromium.googlesource.com/chromium/src/+/1dc8279c01912b090d276d9257c0407e31279a1d

[Taylor_Brandstetter, 2017-06-28 09:47:56]

deadbeef@chromium.org changed reviewers:
+ sergeyu@chromium.org

[Taylor_Brandstetter, 2017-06-28 09:47:56]

PTAL

[Sergey Ulanov, 2017-06-28 17:40:07]

LGTM after my comment is addressed.

https://codereview.chromium.org/2963733002/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/p2p/socket_dispatcher_host.h (right):

https://codereview.chromium.org/2963733002/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/p2p/socket_dispatcher_host.h:133:
base::CreateSequencedTaskRunnerWithTraits(
It's better to put this initialization in the constructor because it calls a
non-trivial function.

[Taylor_Brandstetter, 2017-06-29 11:58:07]

https://codereview.chromium.org/2963733002/diff/1/content/browser/renderer_ho...
File content/browser/renderer_host/p2p/socket_dispatcher_host.h (right):

https://codereview.chromium.org/2963733002/diff/1/content/browser/renderer_ho...
content/browser/renderer_host/p2p/socket_dispatcher_host.h:133:
base::CreateSequencedTaskRunnerWithTraits(
On 2017/06/28 17:40:07, Sergey Ulanov wrote:
> It's better to put this initialization in the constructor because it calls a
> non-trivial function.

Done; that's what I would have done originally, but the example in the
TaskScheduler migration example did it in the header
(https://docs.google.com/presentation/d/191H9hBO0r5pH2JVMeYYV-yrP1175JoSlrZyK5...).
Maybe they just did that so they could fit it on one slide.

[Taylor_Brandstetter, 2017-06-29 11:58:14]

The CQ bit was checked by deadbeef@chromium.org

[Taylor_Brandstetter, 2017-06-29 11:58:14]

The patchset sent to the CQ was uploaded after l-g-t-m from sergeyu@chromium.org
Link to the patchset: https://codereview.chromium.org/2963733002/#ps20001
(title: "Move TaskRunner creation to .cc file.")

[commit-bot: I haz the power, 2017-06-29 11:58:25]

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-06-29 13:07:09]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498737494465660,
"parent_rev": "5637eff43f718e4479c6b107ff7f3628b72b0984", "commit_rev":
"1dc8279c01912b090d276d9257c0407e31279a1d"}

[commit-bot: I haz the power, 2017-06-29 13:07:23]

Description was changed from

==========
Use TaskScheduler instead of PostTask in P2PSocketDispatcherHost.

Using USER_VISIBLE priority, since updating the network list may be
needed to establish a WebRTC connection initiated by the user.

BUG=chromium:667892
==========

to

==========
Use TaskScheduler instead of PostTask in P2PSocketDispatcherHost.

Using USER_VISIBLE priority, since updating the network list may be
needed to establish a WebRTC connection initiated by the user.

BUG=chromium:667892

Review-Url: https://codereview.chromium.org/2963733002
Cr-Commit-Position: refs/heads/master@{#483342}
Committed:
https://chromium.googlesource.com/chromium/src/+/1dc8279c01912b090d276d9257c0...
==========

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

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/1dc8279c01912b090d276d9257c0...