Bug 1148354 - Remove the tests related to the doppler effect from mochitests. r?karlt draft
authorPaul Adenot <paul@paul.cx>
Tue, 17 Jul 2018 10:04:33 +0200
changeset 820845 a9937e58b3e5
parent 820823 ab4dd83bf930
child 820846 738a0b8234d8
push id116959
push userpaul@paul.cx
push dateFri, 20 Jul 2018 14:21:19 +0000
reviewerskarlt
bugs1148354
milestone63.0a1
Bug 1148354 - Remove the tests related to the doppler effect from mochitests. r?karlt MozReview-Commit-ID: 7DZy9ySQrTc
dom/media/test/crashtests/crashtests.list
dom/media/webaudio/test/mochitest.ini
dom/media/webaudio/test/test_AudioListener.html
dom/media/webaudio/test/test_bug867203.html
dom/media/webaudio/test/test_pannerNode.html
--- a/dom/media/test/crashtests/crashtests.list
+++ b/dom/media/test/crashtests/crashtests.list
@@ -100,17 +100,16 @@ load 1411322.html
 load 1450845.html
 load disconnect-wrong-destination.html
 load analyser-channels-1.html
 skip-if(verify&&isDebugBuild&&gtkWidget) load audiocontext-double-suspend.html
 load buffer-source-duration-1.html
 skip-if(verify&&isDebugBuild&&gtkWidget) load buffer-source-ended-1.html
 load buffer-source-resampling-start-1.html
 load buffer-source-slow-resampling-1.html
-skip-if(verify&&isDebugBuild&&gtkWidget) load doppler-1.html
 skip-if(verify&&isDebugBuild&&gtkWidget) HTTP load media-element-source-seek-1.html
 skip-if(verify&&isDebugBuild&&gtkWidget) load offline-buffer-source-ended-1.html
 load oscillator-ended-1.html
 load oscillator-ended-2.html
 skip-if(Android&&AndroidVersion=='22') load video-replay-after-audio-end.html # bug 1315125, bug 1358876
 # This needs to run at the end to avoid leaking busted state into other tests.
 skip-if(Android) load 691096-1.html # Bug 1365451
 load 1236639.html
--- a/dom/media/webaudio/test/mochitest.ini
+++ b/dom/media/webaudio/test/mochitest.ini
@@ -88,17 +88,16 @@ tags=capturestream
 [test_bug839753.html]
 [test_bug845960.html]
 [test_bug856771.html]
 [test_bug866570.html]
 [test_bug866737.html]
 [test_bug867089.html]
 [test_bug867104.html]
 [test_bug867174.html]
-[test_bug867203.html]
 [test_bug875221.html]
 [test_bug875402.html]
 [test_bug894150.html]
 [test_bug956489.html]
 [test_bug964376.html]
 [test_bug966247.html]
 tags=capturestream
 [test_bug972678.html]
--- a/dom/media/webaudio/test/test_AudioListener.html
+++ b/dom/media/webaudio/test/test_AudioListener.html
@@ -8,28 +8,21 @@
 <body>
 <pre id="test">
 <script class="testbody" type="text/javascript">
 
 SimpleTest.waitForExplicitFinish();
 addLoadEvent(function() {
   var context = new AudioContext();
   ok("listener" in context, "AudioContext.listener should exist");
-  ok(Math.abs(context.listener.dopplerFactor - 1.0) < 1e-4, "Correct default doppler factor");
-  ok(Math.abs(context.listener.speedOfSound - 343.3) < 1e-4, "Correct default speed of sound value");
-  context.listener.dopplerFactor = 0.5;
-  ok(Math.abs(context.listener.dopplerFactor - 0.5) < 1e-4, "The doppler factor value can be changed");
-  context.listener.speedOfSound = 400;
-  ok(Math.abs(context.listener.speedOfSound - 400) < 1e-4, "The speed of sound can be changed");
   // The values set by the following cannot be read from script, but the
   // implementation is simple enough, so we just make sure that nothing throws.
   with (context.listener) {
     setPosition(1.0, 1.0, 1.0);
     setOrientation(1.0, 1.0, 1.0, 1.0, 1.0, 1.0);
-    setVelocity(0.5, 1.0, 1.5);
   }
   SimpleTest.finish();
 });
 
 </script>
 </pre>
 </body>
 </html>
deleted file mode 100644
--- a/dom/media/webaudio/test/test_bug867203.html
+++ /dev/null
@@ -1,34 +0,0 @@
-<!DOCTYPE HTML>
-<html>
-<head>
-  <title>Crashtest for bug 867203</title>
-  <script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
-  <link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
-</head>
-<body>
-<pre id="test">
-<script class="testbody" type="text/javascript">
-
-SimpleTest.waitForExplicitFinish();
-addLoadEvent(function() {
-  var ctx = new AudioContext();
-
-  var panner1 = ctx.createPanner();
-  panner1.setVelocity(1, 1, 1);
-  ctx.listener.setVelocity(1, 1, 1);
-  (function() {
-    ctx.createBufferSource().connect(panner1);
-  })();
-  SpecialPowers.forceGC();
-  SpecialPowers.forceCC();
-  ctx.createPanner();
-
-  ok(true, "We did not crash.");
-  SimpleTest.finish();
-});
-
-
-</script>
-</pre>
-</body>
-</html>
--- a/dom/media/webaudio/test/test_pannerNode.html
+++ b/dom/media/webaudio/test/test_pannerNode.html
@@ -50,18 +50,16 @@ addLoadEvent(function() {
   near(panner.positionY.value, 1, "setPosition sets AudioParam properly");
   near(panner.positionZ.value, 1, "setPosition sets AudioParam properly");
 
   panner.setOrientation(0, 1, 0);
   near(panner.orientationX.value, 0, "setOrientation sets AudioParam properly");
   near(panner.orientationY.value, 1, "setOrientation sets AudioParam properly");
   near(panner.orientationZ.value, 0, "setOrientation sets AudioParam properly");
 
-  panner.setVelocity(1, 1, 1);
-
   source.start(0);
   SimpleTest.executeSoon(function() {
     source.stop(0);
     source.disconnect();
     panner.disconnect();
 
     SimpleTest.finish();
   });