Cleaning up peerconnection_unittest.

Cleaning up peerconnection_unittest.

Merging the PeerConnectionTestClientBase and JsepTestClient classes,
since there's no real logical distinction. This should make it slightly
less painful to write new PeerConnection tests.

Committed: https://crrev.com/af1b59cf271854177915342692a78ec0aba61ccd
Cr-Commit-Position: refs/heads/master@{#10292}

[Taylor Brandstetter, 2015-10-14 20:53:52]

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

[Taylor Brandstetter, 2015-10-14 20:53:52]

Guo-wei, since Peter's on vacation would you mind looking at this? It should be
just moving code around (and deleting obsolete code).

[guoweis_webrtc, 2015-10-14 22:00:37]

On 2015/10/14 20:53:52, Taylor Brandstetter wrote:
> Guo-wei, since Peter's on vacation would you mind looking at this? It should
be
> just moving code around (and deleting obsolete code).

this seems simply moving the code around. lgtm.

[pthatcher1, 2015-10-15 05:40:02]

lgtm

[Taylor Brandstetter, 2015-10-15 17:22:24]

The CQ bit was checked by deadbeef@webrtc.org

[commit-bot: I haz the power, 2015-10-15 17:22:35]

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

[commit-bot: I haz the power, 2015-10-15 19:08:47]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-10-15 19:08:57]

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