Test FFT scaling for all orders

Test FFT scaling for all orders

Previously we only tested FFTs up to size 2048, but the AnalyserNode
supports up to 32768.  Test all sizes.

BUG=676722
TEST=realtimeanalyser-fft-scaling.html

Review-Url: https://codereview.chromium.org/2629543005
Cr-Commit-Position: refs/heads/master@{#443461}
Committed: https://chromium.googlesource.com/chromium/src/+/67926a91215c713e851b4fa83bf64e50907ef53e

[Raymond Toy, 2017-01-12 22:12:38]

rtoy@chromium.org changed reviewers:
+ hongchan@chromium.org

[Raymond Toy, 2017-01-12 22:12:38]

PTAL

[hongchan, 2017-01-12 23:26:29]

lgtm with nit

https://codereview.chromium.org/2629543005/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-fft-scaling.html
(right):

https://codereview.chromium.org/2629543005/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-fft-scaling.html:37:
let peakThreshold = [-14.43, -13.56, -13.56, -13.56, -13.56, -13.56, -
This line is wrapped at a weird place. Can we push the last '-' to the next
line?

[Raymond Toy, 2017-01-12 23:33:45]

https://codereview.chromium.org/2629543005/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-fft-scaling.html
(right):

https://codereview.chromium.org/2629543005/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/webaudio/Analyser/realtimeanalyser-fft-scaling.html:37:
let peakThreshold = [-14.43, -13.56, -13.56, -13.56, -13.56, -13.56, -
On 2017/01/12 23:26:29, hongchan wrote:
> This line is wrapped at a weird place. Can we push the last '-' to the next
> line?

Stupid js-beautify.

Done.

[Raymond Toy, 2017-01-12 23:33:53]

The CQ bit was checked by rtoy@chromium.org

[Raymond Toy, 2017-01-12 23:33:54]

The patchset sent to the CQ was uploaded after l-g-t-m from
hongchan@chromium.org
Link to the patchset: https://codereview.chromium.org/2629543005/#ps20001
(title: "Wrap line better")

[commit-bot: I haz the power, 2017-01-12 23:34:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-13 02:10:23]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484264033976110,
"parent_rev": "e317130cc2b2a3b582685541c6a8b8ee1d8deb9c", "commit_rev":
"67926a91215c713e851b4fa83bf64e50907ef53e"}

[commit-bot: I haz the power, 2017-01-13 02:10:54]

Description was changed from

==========
Test FFT scaling for all orders

Previously we only tested FFTs up to size 2048, but the AnalyserNode
supports up to 32768.  Test all sizes.

BUG=676722
TEST=realtimeanalyser-fft-scaling.html
==========

to

==========
Test FFT scaling for all orders

Previously we only tested FFTs up to size 2048, but the AnalyserNode
supports up to 32768.  Test all sizes.

BUG=676722
TEST=realtimeanalyser-fft-scaling.html

Review-Url: https://codereview.chromium.org/2629543005
Cr-Commit-Position: refs/heads/master@{#443461}
Committed:
https://chromium.googlesource.com/chromium/src/+/67926a91215c713e851b4fa83bf6...
==========

[commit-bot: I haz the power, 2017-01-13 02:10:55]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/67926a91215c713e851b4fa83bf6...