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

Unified Diff: third_party/WebKit/LayoutTests/webaudio/resources/scriptprocessornode-testing.js

Issue 2895963003: Apply layout-test-tidy to LayoutTests/webaudio (Closed)
Patch Set: Created 3 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/resources/scriptprocessornode-testing.js
diff --git a/third_party/WebKit/LayoutTests/webaudio/resources/scriptprocessornode-testing.js b/third_party/WebKit/LayoutTests/webaudio/resources/scriptprocessornode-testing.js
index a94c6e959815169a88cf5b0588f877a7583d9be6..bbce612bfa8724330dff119e6febca256e71d259 100644
--- a/third_party/WebKit/LayoutTests/webaudio/resources/scriptprocessornode-testing.js
+++ b/third_party/WebKit/LayoutTests/webaudio/resources/scriptprocessornode-testing.js
@@ -1,84 +1,88 @@
-// For the current implementation of JavaScriptAudioNode, when it works with OfflineAudioContext (which runs much faster
-// than real-time) the event.inputBuffer might be overwrite again before onaudioprocess ever get chance to be called.
-// We carefully arrange the renderLengthInFrames and bufferSize to have exactly the same value to avoid this issue.
+// For the current implementation of JavaScriptAudioNode, when it works with
+// OfflineAudioContext (which runs much faster than real-time) the
+// event.inputBuffer might be overwrite again before onaudioprocess ever get
+// chance to be called. We carefully arrange the renderLengthInFrames and
+// bufferSize to have exactly the same value to avoid this issue.
var renderLengthInFrames = 512;
var bufferSize = 512;
var context;
function createBuffer(context, numberOfChannels, length) {
- var audioBuffer = context.createBuffer(numberOfChannels, length, sampleRate);
+ let audioBuffer = context.createBuffer(numberOfChannels, length, sampleRate);
- fillData(audioBuffer, numberOfChannels, audioBuffer.length);
- return audioBuffer;
+ fillData(audioBuffer, numberOfChannels, audioBuffer.length);
+ return audioBuffer;
}
function processAudioData(event) {
- buffer = event.outputBuffer;
- if (buffer.numberOfChannels != outputChannels)
- testFailed("numberOfOutputChannels doesn't match!");
+ buffer = event.outputBuffer;
+ if (buffer.numberOfChannels != outputChannels)
+ testFailed('numberOfOutputChannels doesn\'t match!');
- if (buffer.length != bufferSize)
- testFailed("length of buffer doesn't match!");
+ if (buffer.length != bufferSize)
+ testFailed('length of buffer doesn\'t match!');
- buffer = event.inputBuffer;
+ buffer = event.inputBuffer;
- var success = checkStereoOnlyData(buffer, inputChannels, buffer.length);
+ let success = checkStereoOnlyData(buffer, inputChannels, buffer.length);
- if (success) {
- testPassed("onaudioprocess was called with correct input data.");
- } else {
- testFailed("onaudioprocess was called with wrong input data.");
- }
+ if (success) {
+ testPassed('onaudioprocess was called with correct input data.');
+ } else {
+ testFailed('onaudioprocess was called with wrong input data.');
+ }
}
function fillData(buffer, numberOfChannels, length) {
- for (var i = 0; i < numberOfChannels; ++i) {
- var data = buffer.getChannelData(i);
-
- for (var j = 0; j < length; ++j)
- if (i < 2)
- data[j] = i * 2 - 1;
- else
- data[j] = 0;
- }
+ for (let i = 0; i < numberOfChannels; ++i) {
+ let data = buffer.getChannelData(i);
+
+ for (let j = 0; j < length; ++j)
+ if (i < 2)
+ data[j] = i * 2 - 1;
+ else
+ data[j] = 0;
+ }
}
-// Both 2 to 8 upmix and 8 to 2 downmix are just directly copy the first two channels and left channels are zeroed.
+// Both 2 to 8 upmix and 8 to 2 downmix are just directly copy the first two
+// channels and left channels are zeroed.
function checkStereoOnlyData(buffer, numberOfChannels, length) {
- for (var i = 0; i < numberOfChannels; ++i) {
- var data = buffer.getChannelData(i);
-
- for (var j = 0; j < length; ++j) {
- if (i < 2) {
- if (data[j] != i * 2 - 1)
- return false;
- } else {
- if (data[j] != 0)
- return false;
- }
- }
+ for (let i = 0; i < numberOfChannels; ++i) {
+ let data = buffer.getChannelData(i);
+
+ for (let j = 0; j < length; ++j) {
+ if (i < 2) {
+ if (data[j] != i * 2 - 1)
+ return false;
+ } else {
+ if (data[j] != 0)
+ return false;
+ }
}
- return true;
+ }
+ return true;
}
-function runJSNodeTest()
-{
- // Create offline audio context.
- context = new OfflineAudioContext(2, renderLengthInFrames, sampleRate);
+function runJSNodeTest() {
+ // Create offline audio context.
+ context = new OfflineAudioContext(2, renderLengthInFrames, sampleRate);
- var sourceBuffer = createBuffer(context, sourceChannels, renderLengthInFrames);
+ let sourceBuffer =
+ createBuffer(context, sourceChannels, renderLengthInFrames);
- var bufferSource = context.createBufferSource();
- bufferSource.buffer = sourceBuffer;
+ let bufferSource = context.createBufferSource();
+ bufferSource.buffer = sourceBuffer;
- var scriptNode = context.createScriptProcessor(bufferSize, inputChannels, outputChannels);
+ let scriptNode =
+ context.createScriptProcessor(bufferSize, inputChannels, outputChannels);
- bufferSource.connect(scriptNode);
- scriptNode.connect(context.destination);
- scriptNode.onaudioprocess = processAudioData;
+ bufferSource.connect(scriptNode);
+ scriptNode.connect(context.destination);
+ scriptNode.onaudioprocess = processAudioData;
- bufferSource.start(0);
- context.oncomplete = finishJSTest;
- context.startRendering();
+ bufferSource.start(0);
+ context.oncomplete = finishJSTest;
+ context.startRendering();
}

Powered by Google App Engine
This is Rietveld 408576698