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

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

Issue 1277443005: Use interpolation for setValueCurveAtTime (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Corrected test thresholds and results Created 5 years, 4 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
« no previous file with comments | « no previous file | LayoutTests/webaudio/audioparam-setValueCurveAtTime-interpolation.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: LayoutTests/webaudio/audioparam-setValueCurveAtTime.html
diff --git a/LayoutTests/webaudio/audioparam-setValueCurveAtTime.html b/LayoutTests/webaudio/audioparam-setValueCurveAtTime.html
index 137945f16a7c22a9c214c429d53e6f8d398e42ce..fa8f9213111fadb5ceaaf44da554478abd053743 100644
--- a/LayoutTests/webaudio/audioparam-setValueCurveAtTime.html
+++ b/LayoutTests/webaudio/audioparam-setValueCurveAtTime.html
@@ -22,9 +22,10 @@ description("Test AudioParam setValueCurveAtTime() functionality.");
// Number of tests to run.
var numberOfTests = 20;
-// Max allowed difference between the rendered data and the expected result. (The error is zero
-// because the rendered curve should really be exactly the same as the reference.)
-var maxAllowedError = 0;
+// Max allowed difference between the rendered data and the expected result. Because of the linear
+// interpolation, the rendered curve isn't exactly the same as the reference. This value is
+// experimentally determined.
+var maxAllowedError = 4.657e-10;
// The amplitude of the sine wave.
var sineAmplitude = 1;
« no previous file with comments | « no previous file | LayoutTests/webaudio/audioparam-setValueCurveAtTime-interpolation.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698