Comment on attachment 695313 [details] [diff] [review] Change mochitests to expect correct behavior Review of attachment 695313 [details] [diff] [review]: ----------------------------------------------------------------- Now that Adam has fixed the issues that these tests were reporting errors on, we need to change todo_is to is. The following test passes but disturbs me. is(test_data.pcLocal.video[0], pcLocal.remoteStreams[0], "Remote stream for local peer is accessible"); is(test_data.pcRemote.video[0], pcRemote.remoteStreams[0], "Remote stream for remote peer is accessible"); This sure looks like it is testing the equality of two objects which have no reason to be equal. At minimum a comment is needed here.

Fix the indentation.

Comment on attachment 695313 [details] [diff] [review] Change mochitests to expect correct behavior While this is the right thing to do, those changes were already part of my patch on bug 796890, which is now on inbound and should have been landed as indicated at the same time with Adam's fix. This patch is not necessary anymore.