Add test for active processing of AudioBufferSourceNode

Test that when an AudioBufferSourceNode stops, the number of channels of its output changes (eventually) to 1 as required when a node is not actively processing anymore. Chrome already supports this notion (via |DisableOutputs()|), so we're just adding a test for this. Bug: 954545 Test: the-audiobuffersourcenode-interface/active-processing.https.html Change-Id: I4fab646df204720cc1efe2e3ae325e60cb22e49b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1643729 Commit-Queue: Raymond Toy <rtoy@chromium.org> Reviewed-by: Hongchan Choi <hongchan@chromium.org> Cr-Commit-Position: refs/heads/master@{#666013}

Add test for active processing of AudioBufferSourceNode
Test that when an AudioBufferSourceNode stops, the number of channels of its output changes (eventually) to 1 as required when a node is not actively processing anymore. Chrome already supports this notion (via |DisableOutputs()|), so we're just adding a test for this. Bug: 954545 Test: the-audiobuffersourcenode-interface/active-processing.https.html Change-Id: I4fab646df204720cc1efe2e3ae325e60cb22e49b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1643729 Commit-Queue: Raymond Toy <rtoy@chromium.org> Reviewed-by: Hongchan Choi <hongchan@chromium.org> Cr-Commit-Position: refs/heads/master@{#666013}
2257aee3 · Raymond Toy · Commit Bot · 54a937ec · 2257aee3 · 2257aee3
Commit 2257aee3 authored Jun 04, 2019 by Raymond Toy Committed by Commit Bot Jun 04, 2019
2 changed files
--- a/third_party/blink/web_tests/external/wpt/webaudio/the-audio-api/the-audiobuffersourcenode-interface/active-processing.https.html
+++ b/third_party/blink/web_tests/external/wpt/webaudio/the-audio-api/the-audiobuffersourcenode-interface/active-processing.https.html
+<!doctype html>
+<html>
+  <head>
+    <title>
+      Test Active Processing for AudioBufferSourceNode
+    </title>
+    <script src="/resources/testharness.js"></script>
+    <script src="/resources/testharnessreport.js"></script>
+    <script src="/webaudio/resources/audit-util.js"></script>
+    <script src="/webaudio/resources/audit.js"></script>
+  </head>
+
+  <body>
+    <script id="layout-test-code">
+      let audit = Audit.createTaskRunner();
+
+      // Arbitrary sample rate. And we only new a few blocks for rendering to
+      // see if things are working.
+      let sampleRate = 8000;
+      let renderLength = 10 * RENDER_QUANTUM_FRAMES;
+
+      // Offline context used for the tests.
+      let context;
+
+      // Number of channels for the AudioBufferSource. Fairly arbitrary, but
+      // should be more than 2.
+      let numberOfChannels = 7;
+
+      // Number of frames in the AudioBuffer.  Fairly arbitrary, but should
+      // probablybe more than one render quantum and significantly less than
+      // |renderLength|.
+      let bufferFrames = 131;
+
+      let filePath =
+          '../the-audioworklet-interface/processors/input-count-processor.js';
+
+      audit.define('Setup graph', (task, should) => {
+        context =
+            new OfflineAudioContext(numberOfChannels, renderLength, sampleRate);
+
+        should(
+            context.audioWorklet.addModule(filePath).then(() => {
+              let buffer = new AudioBuffer({
+                numberOfChannels: numberOfChannels,
+                length: bufferFrames,
+                sampleRate: context.sampleRate
+              });
+
+              src = new AudioBufferSourceNode(context, {buffer: buffer});
+              let counter = new AudioWorkletNode(context, 'counter');
+
+              src.connect(counter).connect(context.destination);
+              src.start();
+            }),
+            'AudioWorklet and graph construction')
+            .beResolved()
+            .then(() => task.done());
+      });
+
+      audit.define('verify count change', (task, should) => {
+        context.startRendering()
+            .then(renderedBuffer => {
+              let output = renderedBuffer.getChannelData(0);
+
+              // Find the first time the number of channels changes to 1.
+              let countChangeIndex = output.findIndex(x => x == 1);
+
+              // Verify that the count did change.  If it didn't there's a bug
+              // in the imploementation, or it takes longer than the render
+              // length to change.  for the latter case, increase the render
+              // length, but it can't be arbitrarily large.  The change needs to
+              // happen at some reasonable time after the source stops.
+              should(countChangeIndex >= 0, 'Number of channels changed')
+                  .beTrue();
+              should(
+                  countChangeIndex, 'Index where input channel count changed')
+                  .beLessThanOrEqualTo(renderLength);
+
+              // Verify the number of channels at the beginning matches the
+              // number of channels in the AudioBuffer.
+              should(
+                  output.slice(0, countChangeIndex),
+                  `Number of channels in input[0:${countChangeIndex - 1}]`)
+                  .beConstantValueOf(numberOfChannels);
+
+              // Verify that after the source has stopped, the number of
+              // channels is 1.
+              should(
+                  output.slice(countChangeIndex),
+                  `Number of channels in input[${countChangeIndex}:]`)
+                  .beConstantValueOf(1);
+            })
+            .then(() => task.done());
+      });
+
+      audit.run();
+    </script>
+
+  </body>
+</html>
--- a/third_party/blink/web_tests/external/wpt/webaudio/the-audio-api/the-audioworklet-interface/processors/input-count-processor.js
+++ b/third_party/blink/web_tests/external/wpt/webaudio/the-audio-api/the-audioworklet-interface/processors/input-count-processor.js
+/**
+ * @class CountProcessor
+ * @extends AudioWorkletProcessor
+ *
+ * This processor class just looks at the number of input channels on the first
+ * input and fills the first output channel with that value.
+ */
+class CountProcessor extends AudioWorkletProcessor {
+  constructor() {
+    super();
+  }
+
+  process(inputs, outputs, parameters) {
+    let input = inputs[0];
+    let output = outputs[0];
+    output[0].fill(input.length);
+
+    return true;
+  }
+}
+
+registerProcessor('counter', CountProcessor);