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

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

Issue 1377903002: Use frames instead of time for running AudioParam timelines. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 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-setValueCurveAtTime.html
diff --git a/third_party/WebKit/LayoutTests/webaudio/audioparam-setValueCurveAtTime.html b/third_party/WebKit/LayoutTests/webaudio/audioparam-setValueCurveAtTime.html
index fa8f9213111fadb5ceaaf44da554478abd053743..ffbc5706f28ce09302ec4f58a01b0201c9407582 100644
--- a/third_party/WebKit/LayoutTests/webaudio/audioparam-setValueCurveAtTime.html
+++ b/third_party/WebKit/LayoutTests/webaudio/audioparam-setValueCurveAtTime.html
@@ -5,6 +5,7 @@
<script src="resources/audio-testing.js"></script>
<script src="resources/audioparam-testing.js"></script>
<script src="../resources/js-test.js"></script>
+<title>Test AudioParam.setValueCurveAtTime</title>
</head>
<body>
@@ -42,13 +43,6 @@ function automation(value, startTime, endTime)
gainNode.gain.setValueCurveAtTime(curve, startTime, endTime - startTime);
}
-// Create a sine wave of the specified duration.
-function createReferenceSineArray(startTime, endTime, startValue, endValue, sampleRate)
-{
- // Ignore |startValue| and |endValue| for the sine wave.
- return createSineWaveArray(endTime - startTime, freqHz, sineAmplitude, sampleRate);
-}
-
function runTest()
{
// The curve of values to use.
@@ -63,7 +57,8 @@ function runTest()
"setValueCurveAtTime()",
maxAllowedError,
createReferenceSineArray,
- 2 * Math.PI * sineAmplitude * freqHz / sampleRate);
+ 2 * Math.PI * sineAmplitude * freqHz / sampleRate,
+ differenceErrorMetric);
}
runTest();

Powered by Google App Engine
This is Rietveld 408576698