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

Unified Diff: third_party/WebKit/LayoutTests/webaudio/BiquadFilter/biquad-getFrequencyResponse.html

Issue 2892803002: Fix layout tests to prevent errors from layout-test-tidy (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/BiquadFilter/biquad-getFrequencyResponse.html
diff --git a/third_party/WebKit/LayoutTests/webaudio/BiquadFilter/biquad-getFrequencyResponse.html b/third_party/WebKit/LayoutTests/webaudio/BiquadFilter/biquad-getFrequencyResponse.html
index 34ec8a65bcab2ed90b3e710491aed88b44a4e386..e36561da5dead53d52e641b4c26b82ae95aded51 100644
--- a/third_party/WebKit/LayoutTests/webaudio/BiquadFilter/biquad-getFrequencyResponse.html
+++ b/third_party/WebKit/LayoutTests/webaudio/BiquadFilter/biquad-getFrequencyResponse.html
@@ -18,55 +18,46 @@ let audit = Audit.createTaskRunner();
// used doesn't matter since we're testing getFrequencyResponse and not the actual filter output.
// The filters are extensively tested in other biquad tests.
-var context;
-
-// The biquad filter node.
-var filter;
-
Raymond Toy 2017/05/19 13:53:38 Was removing context and filter the actual fix her
hongchan 2017/05/19 16:15:29 Yes, because the imported scripts already declared
// The magnitude response of the biquad filter.
-var magResponse;
+let magResponse;
// The phase response of the biquad filter.
-var phaseResponse;
+let phaseResponse;
// Number of frequency samples to take.
-var numberOfFrequencies = 1000;
+let numberOfFrequencies = 1000;
// The filter parameters.
-var filterCutoff = 1000; // Hz.
-var filterQ = 1;
-var filterGain = 5; // Decibels.
+let filterCutoff = 1000; // Hz.
+let filterQ = 1;
+let filterGain = 5; // Decibels.
// The maximum allowed error in the magnitude response.
-var maxAllowedMagError = 5.7e-7;
+let maxAllowedMagError = 5.7e-7;
// The maximum allowed error in the phase response.
-var maxAllowedPhaseError = 4.7e-8;
-
-// The magnitudes and phases of the reference frequency response.
-var magResponse;
-var phaseResponse;
+let maxAllowedPhaseError = 4.7e-8;
// The magnitudes and phases of the reference frequency response.
-var expectedMagnitudes;
-var expectedPhases;
+let expectedMagnitudes;
+let expectedPhases;
// Convert frequency in Hz to a normalized frequency between 0 to 1 with 1 corresponding to the
// Nyquist frequency.
function normalizedFrequency(freqHz, sampleRate)
{
- var nyquist = sampleRate / 2;
+ let nyquist = sampleRate / 2;
return freqHz / nyquist;
}
// Get the filter response at a (normalized) frequency |f| for the filter with coefficients |coef|.
function getResponseAt(coef, f)
{
- var b0 = coef.b0;
- var b1 = coef.b1;
- var b2 = coef.b2;
- var a1 = coef.a1;
- var a2 = coef.a2;
+ let b0 = coef.b0;
+ let b1 = coef.b1;
+ let b2 = coef.b2;
+ let a1 = coef.a1;
+ let a2 = coef.a2;
// H(z) = (b0 + b1 / z + b2 / z^2) / (1 + a1 / z + a2 / z^2)
//
@@ -82,15 +73,15 @@ function getResponseAt(coef, f)
//
// Compute the magnitude and phase from the real and imaginary parts.
- var omega = Math.PI * f;
- var numeratorReal = b0 + b1 * Math.cos(omega) + b2 * Math.cos(2 * omega);
- var numeratorImag = -(b1 * Math.sin(omega) + b2 * Math.sin(2 * omega));
- var denominatorReal = 1 + a1 * Math.cos(omega) + a2 * Math.cos(2 * omega);
- var denominatorImag = -(a1 * Math.sin(omega) + a2 * Math.sin(2 * omega));
+ let omega = Math.PI * f;
+ let numeratorReal = b0 + b1 * Math.cos(omega) + b2 * Math.cos(2 * omega);
+ let numeratorImag = -(b1 * Math.sin(omega) + b2 * Math.sin(2 * omega));
+ let denominatorReal = 1 + a1 * Math.cos(omega) + a2 * Math.cos(2 * omega);
+ let denominatorImag = -(a1 * Math.sin(omega) + a2 * Math.sin(2 * omega));
- var magnitude = Math.sqrt((numeratorReal * numeratorReal + numeratorImag * numeratorImag)
+ let magnitude = Math.sqrt((numeratorReal * numeratorReal + numeratorImag * numeratorImag)
/ (denominatorReal * denominatorReal + denominatorImag * denominatorImag));
- var phase = Math.atan2(numeratorImag, numeratorReal) - Math.atan2(denominatorImag, denominatorReal);
+ let phase = Math.atan2(numeratorImag, numeratorReal) - Math.atan2(denominatorImag, denominatorReal);
if (phase >= Math.PI) {
phase -= 2 * Math.PI;
@@ -105,15 +96,15 @@ function getResponseAt(coef, f)
// given by |frequencies|.
function frequencyResponseReference(filter, frequencies)
{
- var sampleRate = filter.context.sampleRate;
- var normalizedFreq = normalizedFrequency(filter.frequency.value, sampleRate);
- var filterCoefficients = createFilter(filter.type, normalizedFreq, filter.Q.value, filter.gain.value);
+ let sampleRate = filter.context.sampleRate;
+ let normalizedFreq = normalizedFrequency(filter.frequency.value, sampleRate);
+ let filterCoefficients = createFilter(filter.type, normalizedFreq, filter.Q.value, filter.gain.value);
- var magnitudes = [];
- var phases = [];
+ let magnitudes = [];
+ let phases = [];
- for (var k = 0; k < frequencies.length; ++k) {
- var response = getResponseAt(filterCoefficients, normalizedFrequency(frequencies[k], sampleRate));
+ for (let k = 0; k < frequencies.length; ++k) {
+ let response = getResponseAt(filterCoefficients, normalizedFrequency(frequencies[k], sampleRate));
magnitudes.push(response.magnitude);
phases.push(response.phase);
}
@@ -124,11 +115,11 @@ function frequencyResponseReference(filter, frequencies)
// Compute a set of linearly spaced frequencies.
function createFrequencies(nFrequencies, sampleRate)
{
- var frequencies = new Float32Array(nFrequencies);
- var nyquist = sampleRate / 2;
- var freqDelta = nyquist / nFrequencies;
+ let frequencies = new Float32Array(nFrequencies);
+ let nyquist = sampleRate / 2;
+ let freqDelta = nyquist / nFrequencies;
- for (var k = 0; k < nFrequencies; ++k) {
+ for (let k = 0; k < nFrequencies; ++k) {
frequencies[k] = k * freqDelta;
}
@@ -148,7 +139,7 @@ function linearToDecibels(x)
// -1 if none.
function findBadNumber(signal)
{
- for (var k = 0; k < signal.length; ++k) {
+ for (let k = 0; k < signal.length; ++k) {
if (!isValidNumber(signal[k])) {
return k;
}
@@ -160,7 +151,7 @@ function findBadNumber(signal)
// of phases.
function absolutePhaseDifference(x, y)
{
- var diff = Math.abs(x - y);
+ let diff = Math.abs(x - y);
if (diff > Math.PI) {
diff = 2 * Math.PI - diff;
@@ -171,19 +162,19 @@ function absolutePhaseDifference(x, y)
// Compare the frequency response with our expected response.
function compareResponses(should, filter, frequencies, magResponse, phaseResponse)
{
- var expectedResponse = frequencyResponseReference(filter, frequencies);
+ let expectedResponse = frequencyResponseReference(filter, frequencies);
expectedMagnitudes = expectedResponse.magnitudes;
expectedPhases = expectedResponse.phases;
- var n = magResponse.length;
- var badResponse = false;
+ let n = magResponse.length;
+ let badResponse = false;
- var maxMagError = -1;
- var maxMagErrorIndex = -1;
+ let maxMagError = -1;
+ let maxMagErrorIndex = -1;
- var k;
- var hasBadNumber;
+ let k;
+ let hasBadNumber;
hasBadNumber = findBadNumber(magResponse);
badResponse = !should(hasBadNumber >= 0 ? 1 : 0,
@@ -214,7 +205,7 @@ function compareResponses(should, filter, frequencies, magResponse, phaseRespons
.beTrue();
for (k = 0; k < n; ++k) {
- var error = Math.abs(linearToDecibels(magResponse[k]) - linearToDecibels(expectedMagnitudes[k]));
+ let error = Math.abs(linearToDecibels(magResponse[k]) - linearToDecibels(expectedMagnitudes[k]));
if (error > maxMagError) {
maxMagError = error;
maxMagErrorIndex = k;
@@ -227,11 +218,11 @@ function compareResponses(should, filter, frequencies, magResponse, phaseRespons
' dB) of magnitude response at frequency ' +
frequencies[maxMagErrorIndex] + ' Hz')
.beLessThanOrEqualTo(linearToDecibels(maxAllowedMagError));
- var maxPhaseError = -1;
- var maxPhaseErrorIndex = -1;
+ let maxPhaseError = -1;
+ let maxPhaseErrorIndex = -1;
for (k = 0; k < n; ++k) {
- var error = absolutePhaseDifference(phaseResponse[k], expectedPhases[k]);
+ let error = absolutePhaseDifference(phaseResponse[k], expectedPhases[k]);
if (error > maxPhaseError) {
maxPhaseError = error;
maxPhaseErrorIndex = k;
@@ -264,7 +255,7 @@ audit.define({
filter.Q.value = filterQ;
filter.gain.value = filterGain;
- var frequencies = createFrequencies(numberOfFrequencies, context.sampleRate);
+ let frequencies = createFrequencies(numberOfFrequencies, context.sampleRate);
magResponse = new Float32Array(numberOfFrequencies);
phaseResponse = new Float32Array(numberOfFrequencies);

Powered by Google App Engine
This is Rietveld 408576698