third_party/WebKit/LayoutTests/webaudio/biquad-automation.html - Issue 2033503004: Avoid slow AudioParam automation path when possible

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

Issue 2033503004: Avoid slow AudioParam automation path when possible (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Move m_smoothedValue from AudioParam to AudioParamTimeline. Created 4 years, 6 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

« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/webaudio/biquad-automation-expected.txt » ('j') | third_party/WebKit/LayoutTests/webaudio/biquad-automation-expected.txt » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: third_party/WebKit/LayoutTests/webaudio/biquad-automation.html

diff --git a/third_party/WebKit/LayoutTests/webaudio/biquad-automation.html b/third_party/WebKit/LayoutTests/webaudio/biquad-automation.html

index f4da83d4460ff861304fcbd7a90037e9c0510a66..4cf779f46210abe81f706abbeb3fb4769fd3551d 100644

--- a/third_party/WebKit/LayoutTests/webaudio/biquad-automation.html

+++ b/third_party/WebKit/LayoutTests/webaudio/biquad-automation.html

@@ -38,8 +38,9 @@

// A property bag with properties |b0|, |b1|, |b2|, |a1|, |a2|. Each propery is an array

// consisting of the coefficients for the time-varying biquad filter.

function generateFilterCoefficients(filterTypeFunction, parameters, duration) {

+ var renderEndFrame = Math.ceil(renderDuration * sampleRate);

var endFrame = Math.ceil(duration * sampleRate);

- var nCoef = endFrame;

+ var nCoef = renderEndFrame;

var b0 = new Float64Array(nCoef);

var b1 = new Float64Array(nCoef);

var b2 = new Float64Array(nCoef);

@@ -53,7 +54,7 @@

var gains = parameters.gain || [0, 0];

var detunes = parameters.detune || [0, 0];

- for (var frame = 0; frame < endFrame; ++frame) {

+ for (var frame = 0; frame <= endFrame; ++frame) {

// Apply linear ramp at frame |frame|.

var f = linearRamp(frame / sampleRate, freqs[0], freqs[1], 0, duration);

var q = linearRamp(frame / sampleRate, qs[0], qs[1], 0, duration);

@@ -73,6 +74,14 @@

++k;

}

+ // Fill the rest of the arrays with the constant value to the end of

+ // the rendering duration.

+ b0.fill(b0[endFrame], endFrame + 1);

+ b1.fill(b1[endFrame], endFrame + 1);

+ b2.fill(b2[endFrame], endFrame + 1);

+ a1.fill(a1[endFrame], endFrame + 1);

+ a2.fill(a2[endFrame], endFrame + 1);

return {b0: b0, b1: b1, b2: b2, a1: a1, a2: a2};

}

@@ -120,11 +129,13 @@

function createFilterVerifier(filterCreator, threshold, parameters, input, message) {

return function (resultBuffer) {

var actual = resultBuffer.getChannelData(0);

- var coefs = generateFilterCoefficients(filterCreator, parameters, renderDuration);

+ var coefs = generateFilterCoefficients(filterCreator, parameters, renderDuration / 2);

hongchan 2016/06/20 16:14:40 Why half?

Raymond Toy 2016/06/20 16:57:56 Actually, any value less than renderDuration - 128

reference = timeVaryingFilter(input, coefs);

- Should(message, actual).beCloseToArray(reference, threshold);

+ Should(message, actual, {

+ verbose: true

+ }).beCloseToArray(reference, threshold);

};

}

@@ -140,7 +151,7 @@

// the beginning and end of the test where the done is outside the pass band of the filter,

// but high in the center where the tone is near the center of the pass band.

var parameters = {

- freq: [centerFreq - 9*440, centerFreq + 9*440]

+ freq: [centerFreq - 5*440, centerFreq + 5*440]

}

var graph = configureGraph(context, centerFreq);

var f = graph.filter;

@@ -148,10 +159,10 @@

f.type = "bandpass";

f.frequency.setValueAtTime(parameters.freq[0], 0);

- f.frequency.linearRampToValueAtTime(parameters.freq[1], renderDuration);

+ f.frequency.linearRampToValueAtTime(parameters.freq[1], renderDuration / 2);

context.startRendering()

- .then(createFilterVerifier(createBandpassFilter, 5e-5, parameters, b.getChannelData(0),

+ .then(createFilterVerifier(createBandpassFilter, 4.8429e-6, parameters, b.getChannelData(0),

"Output of bandpass filter with frequency automation"))

.then(done);

});

@@ -180,10 +191,10 @@

f.type = "bandpass";

f.frequency.value = parameters.freq[0];

f.Q.setValueAtTime(parameters.Q[0], 0);

- f.Q.linearRampToValueAtTime(parameters.Q[1], renderDuration);

+ f.Q.linearRampToValueAtTime(parameters.Q[1], renderDuration / 2);

context.startRendering()

- .then(createFilterVerifier(createBandpassFilter, 1.4e-6, parameters, b.getChannelData(0),

+ .then(createFilterVerifier(createBandpassFilter, 1.1062e-6, parameters, b.getChannelData(0),

"Output of bandpass filter with Q automation"))

.then(done);

});

@@ -210,10 +221,10 @@

f.type = "lowshelf";

f.frequency.value = parameters.freq[0];

f.gain.setValueAtTime(parameters.gain[0], 0);

- f.gain.linearRampToValueAtTime(parameters.gain[1], renderDuration);

+ f.gain.linearRampToValueAtTime(parameters.gain[1], renderDuration / 2);

context.startRendering()

- .then(createFilterVerifier(createLowShelfFilter, 8e-6, parameters, b.getChannelData(0),

+ .then(createFilterVerifier(createLowShelfFilter, 1.4306e-5, parameters, b.getChannelData(0),

"Output of lowshelf filter with gain automation"))

.then(done);

});

@@ -234,10 +245,10 @@

f.type = "bandpass";

f.frequency.value = parameters.freq[0];

f.detune.setValueAtTime(parameters.detune[0], 0);

- f.detune.linearRampToValueAtTime(parameters.detune[1], renderDuration);

+ f.detune.linearRampToValueAtTime(parameters.detune[1], renderDuration / 2);

context.startRendering()

- .then(createFilterVerifier(createBandpassFilter, 5e-6, parameters, b.getChannelData(0),

+ .then(createFilterVerifier(createBandpassFilter, 2.9535e-5, parameters, b.getChannelData(0),

"Output of bandpass filter with detune automation"))

.then(done);

});

@@ -252,7 +263,7 @@

// Sweep all of the filter parameters. These are pretty much arbitrary.

var parameters = {

- freq: [10000, 100],

+ freq: [8000, 100],

Q: [f.Q.value, .0001],

gain: [f.gain.value, 20],

detune: [2400, -2400]

@@ -267,13 +278,13 @@

f.detune.setValueAtTime(parameters.detune[0], 0);

// Linear ramp each parameter