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

Unified Diff: LayoutTests/webaudio/audiobuffersource-start.html

Issue 723823002: AudioBufferSourceNode loop duration should be the actual duration (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: One item per line Created 5 years, 10 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: LayoutTests/webaudio/audiobuffersource-start.html
diff --git a/LayoutTests/webaudio/audiobuffersource-start.html b/LayoutTests/webaudio/audiobuffersource-start.html
index b3944b8a0a9009116cd227c96a7d7842520c83b0..012c0c46c7dbd71bde80a620f85ae1d1c153e42d 100644
--- a/LayoutTests/webaudio/audiobuffersource-start.html
+++ b/LayoutTests/webaudio/audiobuffersource-start.html
@@ -43,6 +43,11 @@ var tests = [
{ description: "start(when, 9_frames): play with explicit offset past end of buffer",
offsetFrame: 8, durationFrames: 0, renderFrames: 16, playbackRate: 1, expected: [0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0] },
+
+// When the duration exceeds the buffer, just play to the end of the buffer.
+// (This is different from the case when we're looping, which is tested in loop-comprehensive.)
+{ description: "start(when, 0, 15_frames): play with whole buffer, with long duration (clipped)",
+ offsetFrame: 0, durationFrames: 15, renderFrames: 16, playbackRate: 1, expected: [0,1,2,3,4,5,6,7,0,0,0,0,0,0,0,0] },
// Enable test when AudioBufferSourceNode hack is fixed: https://bugs.webkit.org/show_bug.cgi?id=77224
// { description: "start(when, 3_frames, 3_frames): play a middle section with explicit offset and duration",

Powered by Google App Engine
This is Rietveld 408576698