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

Unified Diff: third_party/WebKit/LayoutTests/webaudio/AudioBuffer/audiobuffer-copy-channel.html

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/AudioBuffer/audiobuffer-copy-channel.html
diff --git a/third_party/WebKit/LayoutTests/webaudio/AudioBuffer/audiobuffer-copy-channel.html b/third_party/WebKit/LayoutTests/webaudio/AudioBuffer/audiobuffer-copy-channel.html
index dfd55bdd0e4ec0716465221ca8d40ec7bf65242b..d55903a1b12207210252e17d060b7f20c40dc277 100644
--- a/third_party/WebKit/LayoutTests/webaudio/AudioBuffer/audiobuffer-copy-channel.html
+++ b/third_party/WebKit/LayoutTests/webaudio/AudioBuffer/audiobuffer-copy-channel.html
@@ -1,57 +1,59 @@
-<!doctype html>
+<!DOCTYPE html>
<html>
<head>
+ <title>
+ Test Basic Functionality of AudioBuffer.copyFromChannel and
+ AudioBuffer.copyToChannel
+ </title>
<script src="../../resources/testharness.js"></script>
- <script src="../../resources/testharnessreport.js"></script>
+ <script src="../../resources/testharnessreport.js"></script>
<script src="../resources/audit-util.js"></script>
<script src="../resources/audit.js"></script>
- <title>Test Basic Functionality of AudioBuffer.copyFromChannel and AudioBuffer.copyToChannel</title>
</head>
-
<body>
- <script>
-
-
+ <script id="layout-test-code">
// Define utility routines.
- // Initialize the AudioBuffer |buffer| with a ramp signal on each channel. The ramp starts at
- // channel number + 1.
- function initializeAudioBufferRamp (buffer) {
- for (var c = 0; c < buffer.numberOfChannels; ++c) {
- var d = buffer.getChannelData(c);
- for (var k = 0; k < d.length; ++k) {
+ // Initialize the AudioBuffer |buffer| with a ramp signal on each channel.
+ // The ramp starts at channel number + 1.
+ function initializeAudioBufferRamp(buffer) {
+ for (let c = 0; c < buffer.numberOfChannels; ++c) {
+ let d = buffer.getChannelData(c);
+ for (let k = 0; k < d.length; ++k) {
d[k] = k + c + 1;
}
}
}
- // Create a Float32Array of length |length| and initialize the array to -1.
+ // Create a Float32Array of length |length| and initialize the array to
+ // -1.
function createInitializedF32Array(length) {
- var x = new Float32Array(length);
- for (var k = 0; k < length; ++k) {
+ let x = new Float32Array(length);
+ for (let k = 0; k < length; ++k) {
x[k] = -1;
}
return x;
}
- // Create a Float32Array of length |length| that is initialized to be a ramp starting at 1.
+ // Create a Float32Array of length |length| that is initialized to be a
+ // ramp starting at 1.
function createFloat32RampArray(length) {
- var x = new Float32Array(length);
- for (var k = 0; k < x.length; ++k) {
+ let x = new Float32Array(length);
+ for (let k = 0; k < x.length; ++k) {
x[k] = k + 1;
}
return x;
}
- // Test that the array |x| is a ramp starting at value |start| of length |length|, starting at
- // |startIndex| in the array. |startIndex| is optional and defaults to 0. Any other values
- // must be -1.
- function shouldBeRamp(should, testName, x, startValue, length,
- startIndex) {
- var k;
- var startingIndex = startIndex || 0;
- var expected = Array(x.length);
+ // Test that the array |x| is a ramp starting at value |start| of length
+ // |length|, starting at |startIndex| in the array. |startIndex| is
+ // optional and defaults to 0. Any other values must be -1.
+ function shouldBeRamp(
+ should, testName, x, startValue, length, startIndex) {
+ let k;
+ let startingIndex = startIndex || 0;
+ let expected = Array(x.length);
// Fill the expected array with the correct results.
@@ -70,232 +72,254 @@
expected[k] = -1;
}
- should(x, testName, {
- numberOfArrayLog: 32
- }).beEqualToArray(expected);
+ should(x, testName, {numberOfArrayLog: 32}).beEqualToArray(expected);
}
- var audit = Audit.createTaskRunner();
+ let audit = Audit.createTaskRunner();
- var context = new AudioContext();
- // Temp array for testing exceptions for copyToChannel/copyFromChannel. The length is arbitrary.
- var x = new Float32Array(8);
+ let context = new AudioContext();
+ // Temp array for testing exceptions for copyToChannel/copyFromChannel.
+ // The length is arbitrary.
+ let x = new Float32Array(8);
- // Number of frames in the AudioBuffer for testing. This is pretty arbitrary so choose a
- // fairly small value.
- var bufferLength = 16;
+ // Number of frames in the AudioBuffer for testing. This is pretty
+ // arbitrary so choose a fairly small value.
+ let bufferLength = 16;
- // Number of channels in the AudioBuffer. Also arbitrary, but it should be greater than 1 for
- // test coverage.
- var numberOfChannels = 3;
+ // Number of channels in the AudioBuffer. Also arbitrary, but it should
+ // be greater than 1 for test coverage.
+ let numberOfChannels = 3;
// AudioBuffer that will be used for testing copyFrom and copyTo.
- var buffer = context.createBuffer(numberOfChannels, bufferLength, context.sampleRate);
+ let buffer = context.createBuffer(
+ numberOfChannels, bufferLength, context.sampleRate);
- var initialValues = Array(numberOfChannels);
+ let initialValues = Array(numberOfChannels);
// Initialize things
- audit.define("initialize", (task, should) => {
+ audit.define('initialize', (task, should) => {
// Initialize to -1.
initialValues.fill(-1);
- should(initialValues, "Initialized values")
- .beConstantValueOf(-1)
+ should(initialValues, 'Initialized values').beConstantValueOf(-1)
task.done();
});
// Test that expected exceptions are signaled for copyFrom.
- audit.define("copyFrom-exceptions", (task, should) => {
- should(AudioBuffer.prototype.copyFromChannel,
- "AudioBuffer.prototype.copyFromChannel")
- .exist();
-
- should(() => {
- buffer = context.createBuffer(numberOfChannels, bufferLength,
- context.sampleRate);
+ audit.define('copyFrom-exceptions', (task, should) => {
+ should(
+ AudioBuffer.prototype.copyFromChannel,
+ 'AudioBuffer.prototype.copyFromChannel')
+ .exist();
+
+ should(
+ () => {
+ buffer = context.createBuffer(
+ numberOfChannels, bufferLength, context.sampleRate);
},
- "0: buffer = context.createBuffer(" + numberOfChannels + ", " +
- bufferLength + ", context.sampleRate)")
- .notThrow();
- should(() => {
- buffer.copyFromChannel(null, 0);
- }, "1: buffer.copyFromChannel(null, 0)")
- .throw("TypeError");
+ '0: buffer = context.createBuffer(' + numberOfChannels + ', ' +
+ bufferLength + ', context.sampleRate)')
+ .notThrow();
should(() => {
- buffer.copyFromChannel(context, 0);
- }, "2: buffer.copyFromChannel(context, 0)")
- .throw("TypeError");
+ buffer.copyFromChannel(null, 0);
+ }, '1: buffer.copyFromChannel(null, 0)').throw('TypeError');
should(() => {
- buffer.copyFromChannel(x, -1);
- }, "3: buffer.copyFromChannel(x, -1)")
- .throw("IndexSizeError");
+ buffer.copyFromChannel(context, 0);
+ }, '2: buffer.copyFromChannel(context, 0)').throw('TypeError');
should(() => {
- buffer.copyFromChannel(x, numberOfChannels);
- }, "4: buffer.copyFromChannel(x, " + numberOfChannels + ")")
- .throw("IndexSizeError");;
- should(() => {
- buffer.copyFromChannel(x, 0, -1);
- }, "5: buffer.copyFromChannel(x, 0, -1)")
- .throw("IndexSizeError");
+ buffer.copyFromChannel(x, -1);
+ }, '3: buffer.copyFromChannel(x, -1)').throw('IndexSizeError');
+ should(
+ () => {
+ buffer.copyFromChannel(x, numberOfChannels);
+ },
+ '4: buffer.copyFromChannel(x, ' + numberOfChannels + ')')
+ .throw('IndexSizeError');
+ ;
should(() => {
- buffer.copyFromChannel(x, 0, bufferLength);
- }, "6: buffer.copyFromChannel(x, 0, " + bufferLength + ")")
- .throw("IndexSizeError");
+ buffer.copyFromChannel(x, 0, -1);
+ }, '5: buffer.copyFromChannel(x, 0, -1)').throw('IndexSizeError');
+ should(
+ () => {
+ buffer.copyFromChannel(x, 0, bufferLength);
+ },
+ '6: buffer.copyFromChannel(x, 0, ' + bufferLength + ')')
+ .throw('IndexSizeError');
should(() => {
- buffer.copyFromChannel(x, 3);
- }, "7: buffer.copyFromChannel(x, 3)")
- .throw("IndexSizeError");
+ buffer.copyFromChannel(x, 3);
+ }, '7: buffer.copyFromChannel(x, 3)').throw('IndexSizeError');
if (window.SharedArrayBuffer) {
- var shared_buffer = new Float32Array(new SharedArrayBuffer(32));
- should(() => {
- buffer.copyFromChannel(shared_buffer, 0);
- }, "8: buffer.copyFromChannel(SharedArrayBuffer view, 0)")
- .throw("TypeError");
-
- should(() => {
- buffer.copyFromChannel(shared_buffer, 0, 0);
- }, "9: buffer.copyFromChannel(SharedArrayBuffer view, 0, 0)")
- .throw("TypeError");
+ let shared_buffer = new Float32Array(new SharedArrayBuffer(32));
+ should(
+ () => {
+ buffer.copyFromChannel(shared_buffer, 0);
+ },
+ '8: buffer.copyFromChannel(SharedArrayBuffer view, 0)')
+ .throw('TypeError');
+
+ should(
+ () => {
+ buffer.copyFromChannel(shared_buffer, 0, 0);
+ },
+ '9: buffer.copyFromChannel(SharedArrayBuffer view, 0, 0)')
+ .throw('TypeError');
}
task.done();
});
// Test that expected exceptions are signaled for copyTo.
- audit.define("copyTo-exceptions", (task, should) => {
- should(AudioBuffer.prototype.copyToChannel,
- "AudioBuffer.prototype.copyToChannel")
- .exist();
- should(() => {
- buffer.copyToChannel(null, 0);
- }, "0: buffer.copyToChannel(null, 0)")
- .throw("TypeError");
+ audit.define('copyTo-exceptions', (task, should) => {
+ should(
+ AudioBuffer.prototype.copyToChannel,
+ 'AudioBuffer.prototype.copyToChannel')
+ .exist();
should(() => {
- buffer.copyToChannel(context, 0);
- }, "1: buffer.copyToChannel(context, 0)")
- .throw("TypeError");
+ buffer.copyToChannel(null, 0);
+ }, '0: buffer.copyToChannel(null, 0)').throw('TypeError');
should(() => {
- buffer.copyToChannel(x, -1);
- }, "2: buffer.copyToChannel(x, -1)")
- .throw("IndexSizeError");
+ buffer.copyToChannel(context, 0);
+ }, '1: buffer.copyToChannel(context, 0)').throw('TypeError');
should(() => {
- buffer.copyToChannel(x, numberOfChannels);
- }, "3: buffer.copyToChannel(x, " + numberOfChannels + ")")
- .throw("IndexSizeError");
- should(() => {
- buffer.copyToChannel(x, 0, -1);
- }, "4: buffer.copyToChannel(x, 0, -1)")
- .throw("IndexSizeError");
+ buffer.copyToChannel(x, -1);
+ }, '2: buffer.copyToChannel(x, -1)').throw('IndexSizeError');
+ should(
+ () => {
+ buffer.copyToChannel(x, numberOfChannels);
+ },
+ '3: buffer.copyToChannel(x, ' + numberOfChannels + ')')
+ .throw('IndexSizeError');
should(() => {
- buffer.copyToChannel(x, 0, bufferLength);
- }, "5: buffer.copyToChannel(x, 0, " + bufferLength + ")")
- .throw("IndexSizeError");
+ buffer.copyToChannel(x, 0, -1);
+ }, '4: buffer.copyToChannel(x, 0, -1)').throw('IndexSizeError');
+ should(
+ () => {
+ buffer.copyToChannel(x, 0, bufferLength);
+ },
+ '5: buffer.copyToChannel(x, 0, ' + bufferLength + ')')
+ .throw('IndexSizeError');
should(() => {
- buffer.copyToChannel(x, 3);
- }, "6: buffer.copyToChannel(x, 3)")
- .throw("IndexSizeError");
+ buffer.copyToChannel(x, 3);
+ }, '6: buffer.copyToChannel(x, 3)').throw('IndexSizeError');
if (window.SharedArrayBuffer) {
- var shared_buffer = new Float32Array(new SharedArrayBuffer(32));
- should(() => {
- buffer.copyToChannel(shared_buffer, 0);
- }, "7: buffer.copyToChannel(SharedArrayBuffer view, 0)")
- .throw("TypeError");
-
- should(() => {
- buffer.copyToChannel(shared_buffer, 0, 0);
- }, "8: buffer.copyToChannel(SharedArrayBuffer view, 0, 0)")
- .throw("TypeError");
+ let shared_buffer = new Float32Array(new SharedArrayBuffer(32));
+ should(
+ () => {
+ buffer.copyToChannel(shared_buffer, 0);
+ },
+ '7: buffer.copyToChannel(SharedArrayBuffer view, 0)')
+ .throw('TypeError');
+
+ should(
+ () => {
+ buffer.copyToChannel(shared_buffer, 0, 0);
+ },
+ '8: buffer.copyToChannel(SharedArrayBuffer view, 0, 0)')
+ .throw('TypeError');
}
task.done();
});
// Test copyFromChannel
- audit.define("copyFrom-validate", (task, should) => {
+ audit.define('copyFrom-validate', (task, should) => {
// Initialize the AudioBuffer to a ramp for testing copyFrom.
initializeAudioBufferRamp(buffer);
- // Test copyFrom operation with a short destination array, filling the destination completely.
- for (var c = 0; c < numberOfChannels; ++c) {
- var dst8 = createInitializedF32Array(8);
+ // Test copyFrom operation with a short destination array, filling the
+ // destination completely.
+ for (let c = 0; c < numberOfChannels; ++c) {
+ let dst8 = createInitializedF32Array(8);
buffer.copyFromChannel(dst8, c);
- shouldBeRamp(should, "buffer.copyFromChannel(dst8, " + c + ")", dst8, c + 1, 8)
+ shouldBeRamp(
+ should, 'buffer.copyFromChannel(dst8, ' + c + ')', dst8, c + 1, 8)
}
- // Test copyFrom operation with a short destination array using a non-zero start index that
- // still fills the destination completely.
- for (var c = 0; c < numberOfChannels; ++c) {
- var dst8 = createInitializedF32Array(8);
+ // Test copyFrom operation with a short destination array using a
+ // non-zero start index that still fills the destination completely.
+ for (let c = 0; c < numberOfChannels; ++c) {
+ let dst8 = createInitializedF32Array(8);
buffer.copyFromChannel(dst8, c, 1);
- shouldBeRamp(should, "buffer.copyFromChannel(dst8, " + c + ", 1)", dst8, c + 2, 8)
+ shouldBeRamp(
+ should, 'buffer.copyFromChannel(dst8, ' + c + ', 1)', dst8, c + 2,
+ 8)
}
- // Test copyFrom operation with a short destination array using a non-zero start index that
- // does not fill the destinatiom completely. The extra elements should be unchanged.
- for (var c = 0; c < numberOfChannels; ++c) {
- var dst8 = createInitializedF32Array(8);
- var startInChannel = bufferLength - 5;
+ // Test copyFrom operation with a short destination array using a
+ // non-zero start index that does not fill the destinatiom completely.
+ // The extra elements should be unchanged.
+ for (let c = 0; c < numberOfChannels; ++c) {
+ let dst8 = createInitializedF32Array(8);
+ let startInChannel = bufferLength - 5;
buffer.copyFromChannel(dst8, c, startInChannel);
- shouldBeRamp(should, "buffer.copyFromChannel(dst8, " + c + ", " + startInChannel + ")", dst8,
- c + 1 + startInChannel, bufferLength - startInChannel);
+ shouldBeRamp(
+ should,
+ 'buffer.copyFromChannel(dst8, ' + c + ', ' + startInChannel + ')',
+ dst8, c + 1 + startInChannel, bufferLength - startInChannel);
}
- // Copy operation with the destination longer than the buffer, leaving the trailing elements
- // of the destination untouched.
- for (var c = 0; c < numberOfChannels; ++c) {
- var dst26 = createInitializedF32Array(bufferLength + 10);
+ // Copy operation with the destination longer than the buffer, leaving
+ // the trailing elements of the destination untouched.
+ for (let c = 0; c < numberOfChannels; ++c) {
+ let dst26 = createInitializedF32Array(bufferLength + 10);
buffer.copyFromChannel(dst26, c);
- shouldBeRamp(should, "buffer.copyFromChannel(dst26, " + c + ")", dst26,
- c + 1, bufferLength);
+ shouldBeRamp(
+ should, 'buffer.copyFromChannel(dst26, ' + c + ')', dst26, c + 1,
+ bufferLength);
}
task.done();
});
// Test copyTo
- audit.define("copyTo-validate", (task, should) => {
- // Create a source consisting of a ramp starting at 1, longer than the AudioBuffer
- var src = createFloat32RampArray(bufferLength + 10);
-
- // Test copyTo with AudioBuffer shorter than Float32Array. The AudioBuffer should be
- // completely filled with the Float32Array.
- should(() => {
- buffer = createConstantBuffer(context, bufferLength,
- initialValues);
+ audit.define('copyTo-validate', (task, should) => {
+ // Create a source consisting of a ramp starting at 1, longer than the
+ // AudioBuffer
+ let src = createFloat32RampArray(bufferLength + 10);
+
+ // Test copyTo with AudioBuffer shorter than Float32Array. The
+ // AudioBuffer should be completely filled with the Float32Array.
+ should(
+ () => {
+ buffer =
+ createConstantBuffer(context, bufferLength, initialValues);
},
- "buffer = createConstantBuffer(context, " + bufferLength + ", [" +
- initialValues + "])")
- .notThrow();
+ 'buffer = createConstantBuffer(context, ' + bufferLength + ', [' +
+ initialValues + '])')
+ .notThrow();
- for (var c = 0; c < numberOfChannels; ++c) {
+ for (let c = 0; c < numberOfChannels; ++c) {
buffer.copyToChannel(src, c);
- shouldBeRamp(should, "buffer.copyToChannel(src, " + c + ")",
- buffer.getChannelData(c), 1, bufferLength);
+ shouldBeRamp(
+ should, 'buffer.copyToChannel(src, ' + c + ')',
+ buffer.getChannelData(c), 1, bufferLength);
}
- // Test copyTo with AudioBuffer longer than the Float32Array. The tail of the AudioBuffer
- // should be unchanged.
+ // Test copyTo with AudioBuffer longer than the Float32Array. The tail
+ // of the AudioBuffer should be unchanged.
buffer = createConstantBuffer(context, bufferLength, initialValues);
- var src10 = createFloat32RampArray(10);
- for (var c = 0; c < numberOfChannels; ++c) {
+ let src10 = createFloat32RampArray(10);
+ for (let c = 0; c < numberOfChannels; ++c) {
buffer.copyToChannel(src10, c);
- shouldBeRamp(should, "buffer.copyToChannel(src10, " + c + ")",
- buffer.getChannelData(c), 1, 10);
+ shouldBeRamp(
+ should, 'buffer.copyToChannel(src10, ' + c + ')',
+ buffer.getChannelData(c), 1, 10);
}
- // Test copyTo with non-default startInChannel. Part of the AudioBuffer should filled with
- // the beginning and end sections untouched.
+ // Test copyTo with non-default startInChannel. Part of the AudioBuffer
+ // should filled with the beginning and end sections untouched.
buffer = createConstantBuffer(context, bufferLength, initialValues);
- for (var c = 0; c < numberOfChannels; ++c) {
- var startInChannel = 5;
+ for (let c = 0; c < numberOfChannels; ++c) {
+ let startInChannel = 5;
buffer.copyToChannel(src10, c, startInChannel);
- shouldBeRamp(should, "buffer.copyToChannel(src10, " + c + ", " +
- startInChannel + ")",
- buffer.getChannelData(c), 1, src10.length, startInChannel);
+ shouldBeRamp(
+ should,
+ 'buffer.copyToChannel(src10, ' + c + ', ' + startInChannel + ')',
+ buffer.getChannelData(c), 1, src10.length, startInChannel);
}
task.done();
@@ -303,6 +327,5 @@
audit.run();
</script>
-
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698