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

Unified Diff: LayoutTests/webaudio/offlineaudiocontext-suspend-resume-graph-manipulation.html

Issue 1140723003: Implement suspend() and resume() for OfflineAudioContext (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Added a layout test for synchronous graph manipulation Created 5 years, 6 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/offlineaudiocontext-suspend-resume-graph-manipulation.html
diff --git a/LayoutTests/webaudio/offlineaudiocontext-suspend-resume-graph-manipulation.html b/LayoutTests/webaudio/offlineaudiocontext-suspend-resume-graph-manipulation.html
new file mode 100644
index 0000000000000000000000000000000000000000..abf7d78db7265c852ada24279766f3c18b975e68
--- /dev/null
+++ b/LayoutTests/webaudio/offlineaudiocontext-suspend-resume-graph-manipulation.html
@@ -0,0 +1,77 @@
+<!doctype html>
+<html>
+ <head>
+ <script src="../resources/js-test.js"></script>
+ <script src="resources/compatibility.js"></script>
+ <script src="resources/audio-testing.js"></script>
+ </head>
+
+ <body>
+ <script>
+ description('Test synchronous graph manipulation with OfflineAudioContext.suspend() and OfflineAudioContext.suspend().');
+ window.jsTestIsAsync = true;
+
+ var context;
+ var renderQuantum = 128;
+ var renderDuration = 3;
+
+ // The sample rate is multiple of the rendering quantum, so suspension
+ // times fall in to the render quantum boundary.
+ var sampleRate = renderQuantum * 100;
+
+ context = new OfflineAudioContext(1, sampleRate * renderDuration, sampleRate);
+
+ // Create a constant buffer of 1.0.
+ var constantBuffer = createConstantBuffer(context, 128, 1.0);
+ var constantSource = context.createBufferSource();
+ constantSource.buffer = dcOffsetBuffer;
Raymond Toy 2015/06/16 17:12:23 dcOffsetBuffer? How did this work?
hongchan 2015/06/16 17:34:36 I missed the variable name change. Fixing it.
+ constantSource.loop = true;
+
+ // The audio output from the beginning (0.0 second) to the first suspend
+ // time should be 0.0 because there is no connection to the destination.
+
+ // Suspend at 1 second and activate the source node. The audio output
+ // should be 1.0 from |suspendTime1| to the next suspension.
+ var suspendTime1 = 1;
+ context.suspend(suspendTime1).then(function () {
+ Should('The first suspend time', context.currentTime).beEqualTo(suspendTime1);
+ constantSource.connect(context.destination);
+ constantSource.start();
+ context.resume();
+ });
+
+ // Suspend at 2 seconds and disconnect the node. The audio output should
+ // be 0.0 from |suspendTime2| to the end.
+ var suspendTime2 = 2;
+ context.suspend(suspendTime2).then(function () {
+ Should('The second suspend time', context.currentTime).beEqualTo(suspendTime2);
+ constantSource.disconnect();
+ context.resume();
+ });
+
+ context.startRendering().then(function (buffer) {
+ verifyResult(buffer);
+ finishJSTest();
+ });
+
+ function verifyResult(buffer) {
+ var data = buffer.getChannelData(0);
+
+ // Split the rendered buffer into 3 segments: [0.0 ~ 1.0], [1.0 ~ 2.0]
Raymond Toy 2015/06/16 17:12:23 What does the ~ mean? Did you mean -?
hongchan 2015/06/16 17:34:36 I replace this with the mathematical expression fo
+ // [2.0 ~ 3.0].
+ var subarray0 = data.subarray(0, suspendTime1 * sampleRate);
+ var subarray1 = data.subarray(suspendTime1 * sampleRate, suspendTime2 * sampleRate);
+ var subarray2 = data.subarray(suspendTime2 * sampleRate, renderDuration * sampleRate);
+
+ // Each segment should contain a constant value of 0, 1 and 0
+ // respectively.
+ Should('Buffer [0~12800]', subarray0).beConstantValueOf(0);
+ Should('Buffer [12800~25600]', subarray1).beConstantValueOf(1);
+ Should('Buffer [25600~38400]', subarray2).beConstantValueOf(0);
+ }
+
+ successfullyParsed = true;
+ </script>
+
+ </body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698