commit 22e0814d51ba730cb8a8eb420b1041e8e6e6ffd8
author deadbeef <deadbeef@webrtc.org> Mon Jun 12 21:30:28 2017
committer Commit Bot <commit-bot@chromium.org> Mon Jun 12 21:30:28 2017
tree a7a42af9fb107b1710b6fc1f48b5cd76c0dc4725
parent 36b1a5fcec6270ec4a5bea87b33a49b418a3cb29

Update VirtualSocketServerTest to use a fake clock.

Since this is a test for a fake network, it's only natural that it uses
a fake clock as well. This makes the tests much faster, less flaky, and
lets them be moved out of  "webrtc_nonparallel_tests", since they no
longer have a dependency on any "real" thing (sockets, or time) and
can be run in parallel as easily as any other tests.

As part of this CL, added the fake clock as an argument to
VirtualSocketServer's and TestClient's constructors, since these classes
have methods that wait synchronously for something to occur, and if the
test is using a fake clock, they need to advance it in order to make
progress.

Lastly, added a DCHECK in Thread::ProcessMessages. If called with a
nonzero time while a fake clock is used, it will get stuck in an
infinite loop; a DCHECK is easier to notice than an infinite loop.

BUG=webrtc:7727, webrtc:2409

Review-Url: https://codereview.webrtc.org/2927413002
Cr-Commit-Position: refs/heads/master@{#18544}