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

Unified Diff: third_party/WebKit/LayoutTests/webaudio/audioparam-setValueCurve-exceptions-expected.txt

Issue 2020753002: Throw exception if curve is too short for setValueCurveAtTime (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 7 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-setValueCurve-exceptions-expected.txt
diff --git a/third_party/WebKit/LayoutTests/webaudio/audioparam-setValueCurve-exceptions-expected.txt b/third_party/WebKit/LayoutTests/webaudio/audioparam-setValueCurve-exceptions-expected.txt
index 90c5c45f4277b1462bf886c15b0f0f163f209ede..0dfa3fec3baa5c8fa79354551e069a6f56e88264 100644
--- a/third_party/WebKit/LayoutTests/webaudio/audioparam-setValueCurve-exceptions-expected.txt
+++ b/third_party/WebKit/LayoutTests/webaudio/audioparam-setValueCurve-exceptions-expected.txt
@@ -41,6 +41,11 @@ PASS setValueCurveAtTime(..., 0.0125, 0.0025) did not throw an exception.
PASS setTargetAtTime(1, 0.015000000000000001, 1) did not throw an exception.
PASS setValueCurve with adjoining automation functions allowed as expected.
+PASS setValueCurveAtTime([], 0, 0.01) threw InvalidStateError: Failed to execute 'setValueCurveAtTime' on 'AudioParam': The curve length provided (0) is less than the minimum bound (2)..
hongchan 2016/05/31 21:45:23 Double periods? Is this because the throwing excep
Raymond Toy 2016/05/31 22:29:55 We have this all over. I think the exception outp
+PASS setValueCurveAtTime([1], 0, 0.01) threw InvalidStateError: Failed to execute 'setValueCurveAtTime' on 'AudioParam': The curve length provided (1) is less than the minimum bound (2)..
+PASS setValueCurveAtTime([1,2], 0, 0.01) did not throw an exception.
+PASS Exceptions for curve length correctly handled.
+
PASS successfullyParsed is true
TEST COMPLETE

Powered by Google App Engine
This is Rietveld 408576698