Bug 853858: Move PeerConnectionTest instance reference into global scope. r=jesup, r=hskupin
☠☠ backed out by f7156b703f3d ☠ ☠
authorAdam Roach [:abr] <adam@nostrum.com>
Thu, 04 Apr 2013 17:08:42 -0500
changeset 127790 fddde2d1e9b2df375ec4263849c32f51f1575de3
parent 127789 c680d2e04a83a28f8ec13024982c035aec444e53
child 127791 84d0583cc8cb9259041fa07c28a1c937250a818a
push id24512
push userryanvm@gmail.com
push dateFri, 05 Apr 2013 20:13:49 +0000
treeherdermozilla-central@139b6ba547fa [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjesup, hskupin
bugs853858
milestone23.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 853858: Move PeerConnectionTest instance reference into global scope. r=jesup, r=hskupin
dom/media/tests/mochitest/test_peerConnection_basicAudio.html
dom/media/tests/mochitest/test_peerConnection_basicAudioVideo.html
dom/media/tests/mochitest/test_peerConnection_basicAudioVideoCombined.html
dom/media/tests/mochitest/test_peerConnection_basicVideo.html
dom/media/tests/mochitest/test_peerConnection_bug827843.html
dom/media/tests/mochitest/test_peerConnection_bug840344.html
--- a/dom/media/tests/mochitest/test_peerConnection_basicAudio.html
+++ b/dom/media/tests/mochitest/test_peerConnection_basicAudio.html
@@ -10,17 +10,18 @@
 <body>
 <pre id="test">
 <script type="application/javascript">
   createHTML({
     bug: "796892",
     title: "Basic audio-only peer connection"
   });
 
+  var test;
   runTest(function () {
-    var test = new PeerConnectionTest();
+    test = new PeerConnectionTest();
     test.setMediaConstraints([{audio: true}], [{audio: true}]);
     test.run();
   }, true);
 </script>
 </pre>
 </body>
 </html>
--- a/dom/media/tests/mochitest/test_peerConnection_basicAudioVideo.html
+++ b/dom/media/tests/mochitest/test_peerConnection_basicAudioVideo.html
@@ -11,18 +11,19 @@
 <pre id="test">
 <script type="application/javascript">
 
   createHTML({
     bug: "796890",
     title: "Basic audio/video (separate) peer connection"
   });
 
+  var test;
   runTest(function () {
-    var test = new PeerConnectionTest();
+    test = new PeerConnectionTest();
     test.setMediaConstraints([{audio: true}, {video: true}],
                              [{audio: true}, {video: true}]);
     test.run();
   }, true);
 </script>
 </pre>
 </body>
 </html>
--- a/dom/media/tests/mochitest/test_peerConnection_basicAudioVideoCombined.html
+++ b/dom/media/tests/mochitest/test_peerConnection_basicAudioVideoCombined.html
@@ -11,18 +11,19 @@
 <pre id="test">
 <script type="application/javascript">
 
   createHTML({
     bug: "796890",
     title: "Basic audio/video (combined) peer connection"
   });
 
+  var test;
   runTest(function () {
-    var test = new PeerConnectionTest();
+    test = new PeerConnectionTest();
     test.setMediaConstraints([{audio: true}, {video: true}],
                              [{audio: true}, {video: true}]);
     test.run();
   }, true);
 </script>
 </pre>
 </body>
 </html>
--- a/dom/media/tests/mochitest/test_peerConnection_basicVideo.html
+++ b/dom/media/tests/mochitest/test_peerConnection_basicVideo.html
@@ -10,17 +10,18 @@
 <body>
 <pre id="test">
 <script type="application/javascript">
   createHTML({
     bug: "796888",
     title: "Basic video-only peer connection"
   });
 
+  var test;
   runTest(function () {
-    var test = new PeerConnectionTest();
+    test = new PeerConnectionTest();
     test.setMediaConstraints([{video: true}], [{video: true}]);
     test.run();
   }, true);
 </script>
 </pre>
 </body>
 </html>
--- a/dom/media/tests/mochitest/test_peerConnection_bug827843.html
+++ b/dom/media/tests/mochitest/test_peerConnection_bug827843.html
@@ -74,18 +74,19 @@
         try  { description = test.pcRemote.remoteDescription; } catch (e) { exception = e; }
         ok(exception, "Attempt to access remoteDescription of pcRemote after close throws exception");
 
         test.next();
       }
     ]
   ];
 
+  var test;
   runTest(function () {
-    var test = new PeerConnectionTest();
+    test = new PeerConnectionTest();
     test.setMediaConstraints([{audio: true}], [{audio: true}]);
     test.chain.append(steps);
     test.run();
   }, true);
 </script>
 </pre>
 </body>
 </html>
--- a/dom/media/tests/mochitest/test_peerConnection_bug840344.html
+++ b/dom/media/tests/mochitest/test_peerConnection_bug840344.html
@@ -101,18 +101,19 @@
             is (answerCount, 2, "Answer count is 2.")
             test.next();
           }
         }
       }
     ]
   ];
 
+  var test;
   runTest(function () {
-    var test = new PeerConnectionTest();
+    test = new PeerConnectionTest();
     test.setMediaConstraints([{audio: true, video: true}], [ ]);
     test.chain.replaceAfter("PC_LOCAL_GUM", steps);
     test.run();
   }, true);
 </script>
 </pre>
 </body>
 </html>