Revert of [Chromoting] Add DataChannelManager to manage optional incoming data channels

Revert of [Chromoting] Add DataChannelManager to manage optional incoming data channels (patchset #8 id:320001 of https://codereview.chromium.org/2907073003/ )

Reason for revert:
New unittests are failing:

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.mac%2FMac10.10_Tests%2F19251%2F%2B%2Frecipes%2Fsteps%2Fremoting_unittests_on_Mac-10.10%2F0%2Flogs%2FDataChannelManagerTest.FullMatchWithAsynchronousPipe%2F0

Original issue's description:
> [Chromoting] Add DataChannelManager to manage optional incoming data channels
> 
> DataChannelManager manages a set of factory functions to create
> DataChannelHandler instances to handle named data channels. So once the peer
> creates a new data channel, DataChannelManager can automatically handle it.
> Lifetime of a DataChannelHandler is consistent with the MessagePipe it received:
> it deletes itself once the MessagePipe is closed.
> 
> BUG=650926
> 
> Review-Url: https://codereview.chromium.org/2907073003
> Cr-Commit-Position: refs/heads/master@{#478442}
> Committed: https://chromium.googlesource.com/chromium/src/+/4a15f818c5fdbd1e2c44bf102047ecd9b2c166e2

TBR=sergeyu@chromium.org,joedow@chromium.org,zijiehe@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=650926
Review-Url: https://codereview.chromium.org/2928133005
Cr-Commit-Position: refs/heads/master@{#478485}
Committed: https://chromium.googlesource.com/chromium/src/+/d3b81665f23d2192c448b17698eb89b3f4988ee5

[Devlin, 2017-06-10 01:06:33]

The CQ bit was checked by rdevlin.cronin@chromium.org

[Devlin, 2017-06-10 01:06:34]

Created Revert of [Chromoting] Add DataChannelManager to manage optional
incoming data channels

[commit-bot: I haz the power, 2017-06-10 01:06: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-06-10 01:07:16]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1497056793771970, "parent_rev":
"5e4281c414fd30997e543e95d6480a6f9e89a7ab", "commit_rev":
"d3b81665f23d2192c448b17698eb89b3f4988ee5"}

[commit-bot: I haz the power, 2017-06-10 01:07:29]

Description was changed from

==========
Revert of [Chromoting] Add DataChannelManager to manage optional incoming data
channels (patchset #8 id:320001 of https://codereview.chromium.org/2907073003/ )

Reason for revert:
New unittests are failing:

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.mac%2FMac10.10_...

Original issue's description:
> [Chromoting] Add DataChannelManager to manage optional incoming data channels
> 
> DataChannelManager manages a set of factory functions to create
> DataChannelHandler instances to handle named data channels. So once the peer
> creates a new data channel, DataChannelManager can automatically handle it.
> Lifetime of a DataChannelHandler is consistent with the MessagePipe it
received:
> it deletes itself once the MessagePipe is closed.
> 
> BUG=650926
> 
> Review-Url: https://codereview.chromium.org/2907073003
> Cr-Commit-Position: refs/heads/master@{#478442}
> Committed:
https://chromium.googlesource.com/chromium/src/+/4a15f818c5fdbd1e2c44bf102047...

TBR=sergeyu@chromium.org,joedow@chromium.org,zijiehe@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=650926
==========

to

==========
Revert of [Chromoting] Add DataChannelManager to manage optional incoming data
channels (patchset #8 id:320001 of https://codereview.chromium.org/2907073003/ )

Reason for revert:
New unittests are failing:

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.mac%2FMac10.10_...

Original issue's description:
> [Chromoting] Add DataChannelManager to manage optional incoming data channels
> 
> DataChannelManager manages a set of factory functions to create
> DataChannelHandler instances to handle named data channels. So once the peer
> creates a new data channel, DataChannelManager can automatically handle it.
> Lifetime of a DataChannelHandler is consistent with the MessagePipe it
received:
> it deletes itself once the MessagePipe is closed.
> 
> BUG=650926
> 
> Review-Url: https://codereview.chromium.org/2907073003
> Cr-Commit-Position: refs/heads/master@{#478442}
> Committed:
https://chromium.googlesource.com/chromium/src/+/4a15f818c5fdbd1e2c44bf102047...

TBR=sergeyu@chromium.org,joedow@chromium.org,zijiehe@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=650926

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

[commit-bot: I haz the power, 2017-06-10 01:07:30]

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