Fixing WebRtcBrowserTest.CallAndModifyStream, for new addStream behavior.

content/test/data/media/peerconnection-call.html

Index: content/test/data/media/peerconnection-call.html
diff --git a/content/test/data/media/peerconnection-call.html b/content/test/data/media/peerconnection-call.html
index 836f14dfc0a2ce5561dd743ec49d7ba498809ea2..f84ded58a82b74720b3cc2bc1ebeaa5603d7ad96 100644
--- a/content/test/data/media/peerconnection-call.html
+++ b/content/test/data/media/peerconnection-call.html
@@ -544,8 +544,10 @@
       // When all the above events have occurred- the test pass.
       setAllEventsOccuredHandler(reportTestSuccess);
 
+      gFirstConnection.removeStream(localStream);

[phoglund_chromium, 2015/11/11 07:56:42]

So the idea is that you detach the stream, modify it, and re-add it? If you
don't re-add it, does that mean the tracks are added on a copy of the local
stream, but not the stream in the peer connection? 

Do we also need to modify enableLocalVideo etc, which also do modifications on
local streams?

[Taylor_Brandstetter, 2015/11/11 16:12:27]

Yes, you have the right idea. And once we have addTrack/removeTrack, one could
just use those methods directly for finer control.

enableLocalVideo is fine, because we *will* still listen for changes to the
tracks. Just not changes to the contents of the stream.

       localStream.addTrack(gLocalStream.getAudioTracks()[0]);
       localStream.removeTrack(localStream.getVideoTracks()[0]);
+      gFirstConnection.addStream(localStream);
       negotiate();
     });
   }