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

Unified Diff: third_party/WebKit/LayoutTests/webaudio/AudioListener/audiolistener-automation-position.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/AudioListener/audiolistener-automation-position.html
diff --git a/third_party/WebKit/LayoutTests/webaudio/AudioListener/audiolistener-automation-position.html b/third_party/WebKit/LayoutTests/webaudio/AudioListener/audiolistener-automation-position.html
index aafca1118892e132092fc9b35cfa13ebe18e297c..ea4ab65a433679749d980be30f3e238030c1d51c 100644
--- a/third_party/WebKit/LayoutTests/webaudio/AudioListener/audiolistener-automation-position.html
+++ b/third_party/WebKit/LayoutTests/webaudio/AudioListener/audiolistener-automation-position.html
@@ -1,184 +1,173 @@
-<!doctype html>
+<!DOCTYPE html>
<html>
<head>
+ <title>
+ Test Automation of AudioListener
+ </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 Automation of AudioListener</title>
</head>
-
<body>
- <script>
-
- var sampleRate = 48000;
- // These tests are quite slow, so don't run for many frames. 1024 frames should be enough to
- // demonstrate that automations are working.
- var renderFrames = 1024;
- var renderDuration = renderFrames / sampleRate;
+ <script id="layout-test-code">
+ let sampleRate = 48000;
+ // These tests are quite slow, so don't run for many frames. 1024 frames
+ // should be enough to demonstrate that automations are working.
+ let renderFrames = 1024;
+ let renderDuration = renderFrames / sampleRate;
- // End time of the automation. Fairly arbitrary, but must be less than the render duration so
- // that we can verify that the automation did end at the correct time and value.
- var endTime = 0.75 * renderDuration;
+ // End time of the automation. Fairly arbitrary, but must be less than
+ // the render duration so that we can verify that the automation did end
+ // at the correct time and value.
+ let endTime = 0.75 * renderDuration;
- var audit = Audit.createTaskRunner();
+ let audit = Audit.createTaskRunner();
// Starting position for the panner
- var defaultStartPosition = {
- x: 0,
- y: 0,
- z: 0
- };
+ let defaultStartPosition = {x: 0, y: 0, z: 0};
// Ending position for the panner
- var pannerEndPosition = {
- x: 1000,
- y: 1000,
- z: 1000
- };
+ let pannerEndPosition = {x: 1000, y: 1000, z: 1000};
- // Ending position for the listener. It MUST be the the negative of the pannerEndPosition!
- var listenerEndPosition = {
+ // Ending position for the listener. It MUST be the the negative of the
+ // pannerEndPosition!
+ let listenerEndPosition = {
x: -pannerEndPosition.x,
y: -pannerEndPosition.y,
z: -pannerEndPosition.z
};
- // Test the linear, inverse, and exponential distance models when the AudioListener is moving
- // instead of the panner. We take advantage that motion is relative and create a reference by
- // moving the panner in one direction. The resulting output is the expected result. Then we
- // redo the test, but this time move the listener in the opposite direction. The output of
- // this test is compared against the panner result. They should be the same.
+ // Test the linear, inverse, and exponential distance models when the
+ // AudioListener is moving instead of the panner. We take advantage that
+ // motion is relative and create a reference by moving the panner in one
+ // direction. The resulting output is the expected result. Then we redo
+ // the test, but this time move the listener in the opposite direction.
+ // The output of this test is compared against the panner result. They
+ // should be the same.
- audit.define("linear", (task, should) => {
+ audit.define('linear', (task, should) => {
runTest({
should: should,
startPosition: defaultStartPosition,
endPosition: pannerEndPosition,
- distanceModel: {
- model: "linear",
- rolloff: 1
- }
+ distanceModel: {model: 'linear', rolloff: 1}
}).then(() => task.done());
});
- audit.define("exponential", (task, should) => {
+ audit.define('exponential', (task, should) => {
runTest({
should: should,
startPosition: defaultStartPosition,
endPosition: pannerEndPosition,
- distanceModel: {
- model: "exponential",
- rolloff: 1.5
- }
+ distanceModel: {model: 'exponential', rolloff: 1.5}
}).then(() => task.done());
});
- audit.define("inverse", (task, should) => {
+ audit.define('inverse', (task, should) => {
runTest({
should: should,
startPosition: defaultStartPosition,
endPosition: pannerEndPosition,
- distanceModel: {
- model: "inverse",
- rolloff: 1
- }
+ distanceModel: {model: 'inverse', rolloff: 1}
}).then(() => task.done());
});
audit.run();
-
+
function createGraph() {
- var context = new OfflineAudioContext(2, renderFrames, sampleRate);
+ let context = new OfflineAudioContext(2, renderFrames, sampleRate);
// Stereo source for the panner.
- var source = context.createBufferSource();
+ let source = context.createBufferSource();
source.buffer = createConstantBuffer(context, renderFrames, [1, 2]);
- var panner = context.createPanner();
- panner.panningModel = "equalpower";
+ let panner = context.createPanner();
+ panner.panningModel = 'equalpower';
source.connect(panner);
panner.connect(context.destination);
- return {
- context: context,
- source: source,
- panner: panner
- };
+ return {context: context, source: source, panner: panner};
}
// Run a listener test with the given options.
function runTest(options) {
- var refResult;
-
- return runPannerTest(options).then(function (renderedBuffer) {
- refResult = renderedBuffer;
- }).then(function () {
- // Move the listener in the opposite direction.
- options.endPosition = listenerEndPosition;
- return runListenerTest(options).then(function (renderedBuffer) {
- compareResults(renderedBuffer, refResult, options);
- });
- });
+ let refResult;
+
+ return runPannerTest(options)
+ .then(function(renderedBuffer) {
+ refResult = renderedBuffer;
+ })
+ .then(function() {
+ // Move the listener in the opposite direction.
+ options.endPosition = listenerEndPosition;
+ return runListenerTest(options).then(function(renderedBuffer) {
+ compareResults(renderedBuffer, refResult, options);
+ });
+ });
}
function runPannerTest(options) {
- var graph = createGraph();
+ let graph = createGraph();
return runTestCore(graph, options, graph.panner);
}
function runListenerTest(options) {
- var graph = createGraph();
+ let graph = createGraph();
return runTestCore(graph, options, graph.context.listener);
}
function runTestCore(graph, options, audioOjbect) {
- var {context, source, panner} = graph;
+ let {context, source, panner} = graph;
// Initialize the panner with known values.
panner.distanceModel = options.distanceModel.model;
panner.rolloffFactor = options.distanceModel.rolloff;
- panner.panningModel = "equalpower";
-
- // Automate the location. audioObject must be either a PannerNode or the context's
- // AudioListener.
-
+ panner.panningModel = 'equalpower';
+
+ // Automate the location. audioObject must be either a PannerNode or
+ // the context's AudioListener.
+
audioOjbect.positionX.setValueAtTime(options.startPosition.x, 0);
audioOjbect.positionY.setValueAtTime(options.startPosition.y, 0);
audioOjbect.positionZ.setValueAtTime(options.startPosition.z, 0);
- audioOjbect.positionX.linearRampToValueAtTime(options.endPosition.x, endTime);
- audioOjbect.positionY.linearRampToValueAtTime(options.endPosition.y, endTime);
- audioOjbect.positionZ.linearRampToValueAtTime(options.endPosition.z, endTime);
+ audioOjbect.positionX.linearRampToValueAtTime(
+ options.endPosition.x, endTime);
+ audioOjbect.positionY.linearRampToValueAtTime(
+ options.endPosition.y, endTime);
+ audioOjbect.positionZ.linearRampToValueAtTime(
+ options.endPosition.z, endTime);
- // Start the source, render the graph, and return the resulting promise from rendering.
+ // Start the source, render the graph, and return the resulting promise
+ // from rendering.
source.start();
- return context.startRendering().then(function (resultBuffer) {
+ return context.startRendering().then(function(resultBuffer) {
return resultBuffer;
});
}
function compareResults(actualResult, expectedResult, options) {
- // Compare the output with the reference output from moving the source in the opposite
- // direction.
+ // Compare the output with the reference output from moving the source
+ // in the opposite direction.
- var expectedLeft = expectedResult.getChannelData(0);
- var expectedRight = expectedResult.getChannelData(1);
+ let expectedLeft = expectedResult.getChannelData(0);
+ let expectedRight = expectedResult.getChannelData(1);
- var actualLeft = actualResult.getChannelData(0);
- var actualRight = actualResult.getChannelData(1);
+ let actualLeft = actualResult.getChannelData(0);
+ let actualRight = actualResult.getChannelData(1);
- var prefix = 'Distance model: "' + options.distanceModel.model + '"';
+ let prefix = 'Distance model: "' + options.distanceModel.model + '"';
prefix += ', rolloff: ' + options.distanceModel.rolloff;
- options.should(actualLeft, prefix + ": left channel")
- .beCloseToArray(expectedLeft, 0);
- options.should(actualRight, prefix + ": right channel")
- .beCloseToArray(expectedRight, 0);
+ options.should(actualLeft, prefix + ': left channel')
+ .beCloseToArray(expectedLeft, 0);
+ options.should(actualRight, prefix + ': right channel')
+ .beCloseToArray(expectedRight, 0);
}
</script>
</body>

Powered by Google App Engine
This is Rietveld 408576698