Plumb explicit IPC task runner through to IPC::Channel creation

Plumb explicit IPC task runner through to IPC::Channel creation

In preparation for a follow-up CL, IPC::ChannelMojo needs to
know which TaskRunner it will be used from despite being constructed
on some arbitray other thread (for e.g. ChannelProxy).

This CL plumbs through an IPC TaskRunner to various Channel
instantiation sites to avoid lots of noise in the other CL.

There are no behavioral changes introduced here.

BUG=612500, 619202
Committed: https://crrev.com/a34707ca8e2548fcc804ffaa9571adcba2d024a8
Cr-Commit-Position: refs/heads/master@{#406486}

[Ken Rockot(use gerrit already), 2016-07-19 17:14:27]

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

[commit-bot: I haz the power, 2016-07-19 17:14:56]

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

[Ken Rockot(use gerrit already), 2016-07-19 17:15:36]

Description was changed from

==========
Plumb explicit IPC task runner through to IPC::Channel creation

In preparation for a follow-up CL, IPC::ChannelMojo needs to
know which TaskRunner it will be used from despite being constructed
on some arbitray other thread (for e.g. ChannelProxy).

This CL plumbs through an IPC TaskRunner to various Channel
instantiation sites. There are no behavioral changes introduced
here.

BUG=612500,619202
==========

to

==========
Plumb explicit IPC task runner through to IPC::Channel creation

In preparation for a follow-up CL, IPC::ChannelMojo needs to
know which TaskRunner it will be used from despite being constructed
on some arbitray other thread (for e.g. ChannelProxy).

This CL plumbs through an IPC TaskRunner to various Channel
instantiation sites to avoid lots of noise in the other CL.

There are no behavioral changes introduced here.

BUG=612500,619202
==========

[Ken Rockot(use gerrit already), 2016-07-19 17:15:46]

rockot@chromium.org changed reviewers:
+ jam@chromium.org

[commit-bot: I haz the power, 2016-07-19 17:30:21]

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

[commit-bot: I haz the power, 2016-07-19 17:30:21]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Ken Rockot(use gerrit already), 2016-07-19 17:37:27]

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

[commit-bot: I haz the power, 2016-07-19 17:38:04]

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

[Ken Rockot(use gerrit already), 2016-07-19 17:43:31]

Actually I think I managed to do this without all this crap plumbing. Please
ignore.

[Ken Rockot(use gerrit already), 2016-07-19 19:13:18]

I lied. This needs to be done after all. Please take a look!

[Ken Rockot(use gerrit already), 2016-07-19 21:18:42]

BTW the CL which uses this is https://codereview.chromium.org/2163633003 which
completes Channel- (and ChannelProxy-) associated interface support with proper
FIFO.

[jam, 2016-07-19 21:40:08]

lgtm, with addition of dcheck in channelproxy that the task runner it gets is
the same one that the channel gets.

[Ken Rockot(use gerrit already), 2016-07-19 22:04:49]

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

[Ken Rockot(use gerrit already), 2016-07-19 22:04:54]

done

[commit-bot: I haz the power, 2016-07-19 22:05:52]

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

[Ken Rockot(use gerrit already), 2016-07-19 23:03:10]

The CQ bit was unchecked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2016-07-19 23:03:14]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2016-07-19 23:03:15]

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

[commit-bot: I haz the power, 2016-07-19 23:03:40]

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, 2016-07-20 01:03:30]

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

[commit-bot: I haz the power, 2016-07-20 01:03:31]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Ken Rockot(use gerrit already), 2016-07-20 02:36:58]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2016-07-20 02:37:15]

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, 2016-07-20 04:28:47]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-07-20 04:28:53]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-20 04:30:55]

Description was changed from

==========
Plumb explicit IPC task runner through to IPC::Channel creation

In preparation for a follow-up CL, IPC::ChannelMojo needs to
know which TaskRunner it will be used from despite being constructed
on some arbitray other thread (for e.g. ChannelProxy).

This CL plumbs through an IPC TaskRunner to various Channel
instantiation sites to avoid lots of noise in the other CL.

There are no behavioral changes introduced here.

BUG=612500,619202
==========

to

==========
Plumb explicit IPC task runner through to IPC::Channel creation

In preparation for a follow-up CL, IPC::ChannelMojo needs to
know which TaskRunner it will be used from despite being constructed
on some arbitray other thread (for e.g. ChannelProxy).

This CL plumbs through an IPC TaskRunner to various Channel
instantiation sites to avoid lots of noise in the other CL.

There are no behavioral changes introduced here.

BUG=612500,619202

Committed: https://crrev.com/a34707ca8e2548fcc804ffaa9571adcba2d024a8
Cr-Commit-Position: refs/heads/master@{#406486}
==========

[commit-bot: I haz the power, 2016-07-20 04:30:56]

Patchset 3 (id:??) landed as
https://crrev.com/a34707ca8e2548fcc804ffaa9571adcba2d024a8
Cr-Commit-Position: refs/heads/master@{#406486}