Remove VirtualSocketServer's dependency on PhysicalSocketServer.

Remove VirtualSocketServer's dependency on PhysicalSocketServer.

The only thing the physical socket server was used for was
"Wait"/"WakeUp", but it could be replaced by a simple rtc::Event.

So, removing this dependency makes things less confusing; the fact that
VirtualSocketServer takes a PhysicalSocketServer may lead someone to
think it uses real sockets internally, when it doesn't.

BUG=None

Review-Url: https://codereview.webrtc.org/2883313003
Cr-Commit-Position: refs/heads/master@{#18172}
Committed: https://chromium.googlesource.com/external/webrtc/+/98e186c71c10fa7d221fa82391865be1adfdff72

[Taylor Brandstetter, 2017-05-16 17:39:37]

The CQ bit was checked by deadbeef@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-16 17:39:39]

Dry run: CQ is trying da patch.

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

[Taylor Brandstetter, 2017-05-16 17:39:59]

deadbeef@webrtc.org changed reviewers:
+ pthatcher@webrtc.org

[commit-bot: I haz the power, 2017-05-16 18:45:10]

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

[commit-bot: I haz the power, 2017-05-16 18:45:10]

Dry run: This issue passed the CQ dry run.

[pthatcher1, 2017-05-17 00:30:07]

lgtm  

Wow, that's nice.

[Taylor Brandstetter, 2017-05-17 00:56:59]

The CQ bit was checked by deadbeef@webrtc.org

[commit-bot: I haz the power, 2017-05-17 00:57:10]

CQ is trying da patch.

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

[commit-bot: I haz the power, 2017-05-17 01:00:07]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494982619586980, "parent_rev":
"73c1234f6aee9ec75074348b8eb3d346f8b64049", "commit_rev":
"98e186c71c10fa7d221fa82391865be1adfdff72"}

[commit-bot: I haz the power, 2017-05-17 01:00:12]

Description was changed from

==========
Remove VirtualSocketServer's dependency on PhysicalSocketServer.

The only thing the physical socket server was used for was
"Wait"/"WakeUp", but it could be replaced by a simple rtc::Event.

So, removing this dependency makes things less confusing; the fact that
VirtualSocketServer takes a PhysicalSocketServer may lead someone to
think it uses real sockets internally, when it doesn't.

BUG=None
==========

to

==========
Remove VirtualSocketServer's dependency on PhysicalSocketServer.

The only thing the physical socket server was used for was
"Wait"/"WakeUp", but it could be replaced by a simple rtc::Event.

So, removing this dependency makes things less confusing; the fact that
VirtualSocketServer takes a PhysicalSocketServer may lead someone to
think it uses real sockets internally, when it doesn't.

BUG=None

Review-Url: https://codereview.webrtc.org/2883313003
Cr-Commit-Position: refs/heads/master@{#18172}
Committed:
https://chromium.googlesource.com/external/webrtc/+/98e186c71c10fa7d221fa8239...
==========

[commit-bot: I haz the power, 2017-05-17 01:00:13]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/98e186c71c10fa7d221fa8239...