Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(78)

Unified Diff: third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-setTargetAtTime.html

Issue 2657033002: Convert AudioParam tests to testharness and new Audit (Closed)
Patch Set: clang-format the JS code. Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-setTargetAtTime.html
diff --git a/third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-setTargetAtTime.html b/third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-setTargetAtTime.html
index 0fd4d74d8bcb263706efafc66e6fcaf0f3a43e3d..e93b9d1dda20ac3934e233135adc4430626689d4 100644
--- a/third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-setTargetAtTime.html
+++ b/third_party/WebKit/LayoutTests/webaudio/AudioParam/audioparam-setTargetAtTime.html
@@ -1,25 +1,24 @@
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
-<script src="../../resources/js-test.js"></script>
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
<script src="../resources/audit-util.js"></script>
-<script src="../resources/audio-testing.js"></script>
+<script src="../resources/audit.js"></script>
<script src="../resources/audioparam-testing.js"></script>
<title>Test AudioParam.setTargetAtTime</title>
</head>
<body>
-<div id="description"></div>
-<div id="console"></div>
-
<script>
-description("Test AudioParam setTargetAtTime() functionality.");
+let audit = Audit.createTaskRunner();
-// Play a long DC signal out through an AudioGainNode, and call setValueAtTime() and
-// setTargetAtTime at regular intervals to set the starting value and the target
-// value. Each time interval has a ramp with a different starting and target value so that
-// there is a discontinuity at each time interval boundary. The discontinuity is for testing
-// timing. Also, we alternate between an increasing and decreasing ramp for each interval.
+// Play a long DC signal out through an AudioGainNode, and call setValueAtTime()
+// and setTargetAtTime at regular intervals to set the starting value and the
+// target value. Each time interval has a ramp with a different starting and
+// target value so that there is a discontinuity at each time interval boundary.
+// The discontinuity is for testing timing. Also, we alternate between an
+// increasing and decreasing ramp for each interval.
// Number of tests to run.
var numberOfTests = 100;
@@ -31,32 +30,24 @@ var maxAllowedError = 1.6953e-6;
var initialValue = 100;
// Set the gain node value to the specified value at the specified time.
-function setValue(value, time)
-{
- gainNode.gain.setValueAtTime(value, time);
+function setValue(value, time) {
+ gainNode.gain.setValueAtTime(value, time);
}
-// Generate an exponential approach starting at |startTime| with a target value of |value|.
-function automation(value, startTime, endTime)
-{
+// Generate an exponential approach starting at |startTime| with a target value
+// of |value|.
+function automation(value, startTime, endTime){
// endTime is not used for setTargetAtTime.
- gainNode.gain.setTargetAtTime(value, startTime, timeConstant)
-}
-
-function runTest()
-{
- createAudioGraphAndTest(numberOfTests,
- initialValue,
- setValue,
- automation,
- "setTargetAtTime()",
- maxAllowedError,
- createExponentialApproachArray);
-}
+ gainNode.gain.setTargetAtTime(value, startTime, timeConstant)}
-runTest();
-successfullyParsed = true;
+audit.define('test', function(task, should) {
+ task.describe('AudioParam setTargetAtTime() functionality.');
+ createAudioGraphAndTest(
+ task, should, numberOfTests, initialValue, setValue, automation,
+ 'setTargetAtTime()', maxAllowedError, createExponentialApproachArray);
+});
+audit.run();
</script>
</body>

Powered by Google App Engine
This is Rietveld 408576698