Add support for Attachment Brokering of IPC::Channels on multiple threads.

Add support for Attachment Brokering of IPC::Channels on multiple threads.

Attachment brokering makes the assumption that there's a single thread
on which IO related to IPC::Channels occurs. This assumption is violated
when the flag --single-process is set.

BUG=590213
Committed: https://crrev.com/a91d0513fa16b2e8c48acbaf7a5c18659ab10d07
Cr-Commit-Position: refs/heads/master@{#382431}

[erikchen, 2016-02-27 02:39:01]

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

[commit-bot: I haz the power, 2016-02-27 02:40:39]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1739203004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1739203004/1

[commit-bot: I haz the power, 2016-02-27 03:19:57]

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

[commit-bot: I haz the power, 2016-02-27 03:19:58]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[erikchen, 2016-02-29 05:19:58]

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

[commit-bot: I haz the power, 2016-02-29 05:20:05]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1739203004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1739203004/20001

[commit-bot: I haz the power, 2016-02-29 06:04:03]

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

[commit-bot: I haz the power, 2016-02-29 06:04:04]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[erikchen, 2016-02-29 17:44:48]

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

[commit-bot: I haz the power, 2016-02-29 17:45:02]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1739203004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1739203004/40001

[commit-bot: I haz the power, 2016-02-29 19:22:15]

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

[commit-bot: I haz the power, 2016-02-29 19:22:16]

Dry run: This issue passed the CQ dry run.

[erikchen, 2016-03-16 21:43:26]

Description was changed from

==========
Add support for use of IPC::Channels on multiple threads, which occurs when the
flag --single-process is set.

BUG=
==========

to

==========
Add support for use of IPC::Channels on multiple threads, which occurs when the
flag --single-process is set.

BUG=590213
==========

[erikchen, 2016-03-16 21:44:29]

Description was changed from

==========
Add support for use of IPC::Channels on multiple threads, which occurs when the
flag --single-process is set.

BUG=590213
==========

to

==========
Add support for Attachment Brokering of IPC::Channels on multiple threads.

Attachment brokering makes the assumption that there's a single thread on which
IO related to IPC::Channels occurs. This assumption is violated when the flag
--single-process is set.

BUG=590213
==========

[erikchen, 2016-03-16 21:44:39]

Description was changed from

==========
Add support for Attachment Brokering of IPC::Channels on multiple threads.

Attachment brokering makes the assumption that there's a single thread on which
IO related to IPC::Channels occurs. This assumption is violated when the flag
--single-process is set.

BUG=590213
==========

to

==========
Add support for Attachment Brokering of IPC::Channels on multiple threads.

Attachment brokering makes the assumption that there's a single thread on which
IO related to IPC::Channels occurs.
 This assumption is violated when the flag --single-process is set.

BUG=590213
==========

[erikchen, 2016-03-16 21:44:52]

Description was changed from

==========
Add support for Attachment Brokering of IPC::Channels on multiple threads.

Attachment brokering makes the assumption that there's a single thread on which
IO related to IPC::Channels occurs.
 This assumption is violated when the flag --single-process is set.

BUG=590213
==========

to

==========
Add support for Attachment Brokering of IPC::Channels on multiple threads.

Attachment brokering makes the assumption that there's a single thread 
on which IO related to IPC::Channels occurs. This assumption is violated
 when the flag --single-process is set.

BUG=590213
==========

[erikchen, 2016-03-16 21:44:58]

Description was changed from

==========
Add support for Attachment Brokering of IPC::Channels on multiple threads.

Attachment brokering makes the assumption that there's a single thread 
on which IO related to IPC::Channels occurs. This assumption is violated
 when the flag --single-process is set.

BUG=590213
==========

to

==========
Add support for Attachment Brokering of IPC::Channels on multiple threads.

Attachment brokering makes the assumption that there's a single thread 
on which IO related to IPC::Channels occurs. This assumption is violated
when the flag --single-process is set.

BUG=590213
==========

[erikchen, 2016-03-16 21:45:10]

erikchen@chromium.org changed reviewers:
+ tsepez@chromium.org

[erikchen, 2016-03-16 21:45:11]

tsepez: Please review.

[Tom Sepez, 2016-03-16 22:16:23]

lgtm

[erikchen, 2016-03-16 22:23:11]

erikchen@chromium.org changed reviewers:
+ avi@chromium.org

[erikchen, 2016-03-16 22:23:12]

avi: Please review content/

[erikchen, 2016-03-16 23:47:01]

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

[erikchen, 2016-03-16 23:47:02]

sergeyu: Please review.

[Avi (use Gerrit), 2016-03-17 02:22:45]

lgtm

[Sergey Ulanov, 2016-03-17 23:41:56]

https://codereview.chromium.org/1739203004/diff/40001/remoting/host/win/wts_s...
File remoting/host/win/wts_session_process_delegate.cc (right):

https://codereview.chromium.org/1739203004/diff/40001/remoting/host/win/wts_s...
remoting/host/win/wts_session_process_delegate.cc:430: channel_.get(),
caller_task_runner_);
the channel_ is initialized with io_task_runner_ above. Should io_task_runner be
passed here as well?

[erikchen, 2016-03-19 00:52:46]

https://codereview.chromium.org/1739203004/diff/40001/remoting/host/win/wts_s...
File remoting/host/win/wts_session_process_delegate.cc (right):

https://codereview.chromium.org/1739203004/diff/40001/remoting/host/win/wts_s...
remoting/host/win/wts_session_process_delegate.cc:430: channel_.get(),
caller_task_runner_);
On 2016/03/17 23:41:56, Sergey Ulanov wrote:
> the channel_ is initialized with io_task_runner_ above. Should io_task_runner
be
> passed here as well?

yes, done.

[Sergey Ulanov, 2016-03-21 19:58:53]

lgtm

[erikchen, 2016-03-21 21:57:10]

The CQ bit was checked by erikchen@chromium.org

[erikchen, 2016-03-21 21:57:11]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
avi@chromium.org, sergeyu@chromium.org
Link to the patchset: https://codereview.chromium.org/1739203004/#ps80001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-03-21 21:57:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1739203004/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1739203004/80001

[erikchen, 2016-03-21 22:30:27]

Patchset #6 (id:100001) has been deleted

[erikchen, 2016-03-21 22:30:33]

The CQ bit was checked by erikchen@chromium.org

[erikchen, 2016-03-21 22:30:33]

The patchset sent to the CQ was uploaded after l-g-t-m from tsepez@chromium.org,
sergeyu@chromium.org, avi@chromium.org
Link to the patchset: https://codereview.chromium.org/1739203004/#ps80001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-03-21 22:30:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1739203004/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1739203004/80001

[commit-bot: I haz the power, 2016-03-21 23:19:48]

Description was changed from

==========
Add support for Attachment Brokering of IPC::Channels on multiple threads.

Attachment brokering makes the assumption that there's a single thread 
on which IO related to IPC::Channels occurs. This assumption is violated
when the flag --single-process is set.

BUG=590213
==========

to

==========
Add support for Attachment Brokering of IPC::Channels on multiple threads.

Attachment brokering makes the assumption that there's a single thread 
on which IO related to IPC::Channels occurs. This assumption is violated
when the flag --single-process is set.

BUG=590213
==========

[commit-bot: I haz the power, 2016-03-21 23:19:49]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-03-21 23:21:02]

Description was changed from

==========
Add support for Attachment Brokering of IPC::Channels on multiple threads.

Attachment brokering makes the assumption that there's a single thread 
on which IO related to IPC::Channels occurs. This assumption is violated
when the flag --single-process is set.

BUG=590213
==========

to

==========
Add support for Attachment Brokering of IPC::Channels on multiple threads.

Attachment brokering makes the assumption that there's a single thread
on which IO related to IPC::Channels occurs. This assumption is violated
when the flag --single-process is set.

BUG=590213

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

[commit-bot: I haz the power, 2016-03-21 23:21:04]

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