Convert RTCPeerConnection-AddRemoveStream tests from js-test to testharness.js

Convert RTCPeerConnection-AddRemoveStream tests from js-test to testharness.js

BUG=614963
Committed: https://crrev.com/a6a1bca0b0b021cd18b0f4fdb2b557d573d08da8
Cr-Commit-Position: refs/heads/master@{#415705}

[jeffcarp, 2016-08-26 23:55:08]

jeffcarp@chromium.org changed reviewers:
+ guidou@chromium.org, qyearsley@chromium.org

[jeffcarp, 2016-08-26 23:56:47]

On 2016/08/26 at 23:55:08, jeffcarp wrote:
> 

I tried something different in this one - I wrapped everything up into one test
using callbacks. I might be easier or harder to read depending on your opinion,
please let me know!

Also - this PR is inconsistent from the first one with respect to declaring and
accessing global variables. It seems weird to refer to `pc` a bunch of times via
`window.pc`, so I want to revisit that decision.

[jeffcarp, 2016-08-26 23:58:43]

On 2016/08/26 at 23:56:47, jeffcarp wrote:
> On 2016/08/26 at 23:55:08, jeffcarp wrote:
> > 
> 
> I tried something different in this one - I wrapped everything up into one
test using callbacks. I might be easier or harder to read depending on your
opinion, please let me know!
> 
> Also - this PR is inconsistent from the first one with respect to declaring
and accessing global variables. It seems weird to refer to `pc` a bunch of times
via `window.pc`, so I want to revisit that decision.

The convention in WPT seems to be consistent with the way this file was written
-- using `var` at the top scope to declare global variables and accessing them
without prefixing `window.` (example:
https://github.com/w3c/web-platform-tests/blob/dccaa413ec1107e87b72245aa5ce83...)

[Guido Urdaneta, 2016-08-29 10:02:40]

lgtm

[jeffcarp, 2016-08-31 17:28:53]

The CQ bit was checked by jeffcarp@chromium.org

[commit-bot: I haz the power, 2016-08-31 17:29:24]

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, 2016-08-31 18:56:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-31 18:57:26]

Description was changed from

==========
Convert RTCPeerConnection-AddRemoveStream tests from js-test to testharness.js

BUG=614963
==========

to

==========
Convert RTCPeerConnection-AddRemoveStream tests from js-test to testharness.js

BUG=614963

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

[commit-bot: I haz the power, 2016-08-31 18:57:27]

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