Bug 1336098 - When the input is silent, AnalyserNode.getFloatFrequencyData should put -Infinity in the buffer. r=pehrsons

This commit is contained in:
Paul Adenot 2017-02-02 17:13:13 +01:00
Родитель eba0707fb7
Коммит 0ac29668f5
3 изменённых файлов: 55 добавлений и 1 удалений

Просмотреть файл

@ -245,7 +245,9 @@ AnalyserNode::GetFloatFrequencyData(const Float32Array& aArray)
size_t length = std::min(size_t(aArray.Length()), mOutputBuffer.Length());
for (size_t i = 0; i < length; ++i) {
buffer[i] = WebAudioUtils::ConvertLinearToDecibels(mOutputBuffer[i], mMinDecibels);
buffer[i] =
WebAudioUtils::ConvertLinearToDecibels(mOutputBuffer[i],
-std::numeric_limits<float>::infinity());
}
}

Просмотреть файл

@ -33,6 +33,7 @@ support-files =
[test_analyserNodeOutput.html]
[test_analyserNodePassThrough.html]
[test_analyserNodeWithGain.html]
[test_analyserNodeMinimum.html]
[test_AudioBuffer.html]
[test_audioBufferSourceNode.html]
[test_audioBufferSourceNodeEnded.html]

Просмотреть файл

@ -0,0 +1,51 @@
<!DOCTYPE HTML>
<html>
<head>
<title>Test AnalyserNode when the input is silent</title>
<script type="text/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
<script type="text/javascript" src="webaudio.js"></script>
<link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css" />
</head>
<body>
<pre id="test">
<script class="testbody" type="text/javascript">
SimpleTest.waitForExplicitFinish();
addLoadEvent(function() {
var ac = new AudioContext();
var analyser = ac.createAnalyser();
var constant = ac.createConstantSource();
var sp = ac.createScriptProcessor(2048, 1, 0);
constant.offset.value = 0.0;
constant.connect(analyser)
.connect(ac.destination);
constant.connect(sp);
var buf = new Float32Array(analyser.frequencyBinCount);
var iteration_count = 10;
sp.onaudioprocess = function() {
analyser.getFloatFrequencyData(buf);
var correct = true;
for (var i = 0; i < buf.length; i++) {
correct &= buf[i] == -Infinity;
}
ok(correct, "silent input process -Infinity in decibel bins");
if(!(iteration_count--)) {
sp.onaudioprocess = null;
constant.stop();
ac.close();
SimpleTest.finish();
}
}
constant.start();
});
</script>
</pre>
</body>
</html>