AnalyserNode.channelCount should be 1

AnalyserNode.channelCount should be 1

The spec says channelCount should be 1.

Also change the default number of output channels to be 1 (was 2).
This reduces complexity of any down-stream nodes so that they will
process mono unless the input to the AnalyserNode has more than one
channel.  Without this, all down-stream nodes will see a stereo input,
doubling processing costs.  This change should not be user-detectable
in any way.

BUG=706610
TEST=constructor/analyser.html

Review-Url: https://codereview.chromium.org/2809923004
Cr-Commit-Position: refs/heads/master@{#466787}
Committed: https://chromium.googlesource.com/chromium/src/+/2dbd3497a559c2227a37a4e108f6380394c049a4

[Raymond Toy, 2017-04-24 16:07:32]

Description was changed from

==========
AnalyserNode.channelCount should be 1

The spec says channelCount should be 1.

Also change the default number of output channels to be 1 (was 2).
This reduces complexity of any down-stream nodes so that they will
process mono unless the input to the AnalyserNode has more than one
channel.  Without this, all down-stream nodes will see a stereo input,
doubling processing costs.  This change should be user-detectable in
any way.

BUG=706610
TEST=constructor/analyser.html
==========

to

==========
AnalyserNode.channelCount should be 1

The spec says channelCount should be 1.

Also change the default number of output channels to be 1 (was 2).
This reduces complexity of any down-stream nodes so that they will
process mono unless the input to the AnalyserNode has more than one
channel.  Without this, all down-stream nodes will see a stereo input,
doubling processing costs.  This change should not be user-detectable
in any way.

BUG=706610
TEST=constructor/analyser.html
==========

[Raymond Toy, 2017-04-24 16:17:20]

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

[Raymond Toy, 2017-04-24 16:17:21]

PTAL

[hongchan, 2017-04-24 20:59:00]

lgtm

[Raymond Toy, 2017-04-24 21:06:07]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-04-24 21:06:43]

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-04-24 22:06:38]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493067967422920,
"parent_rev": "9a16c13f62099154b115bf2e3464efb075b9d3f9", "commit_rev":
"2dbd3497a559c2227a37a4e108f6380394c049a4"}

[commit-bot: I haz the power, 2017-04-24 22:06:51]

Description was changed from

==========
AnalyserNode.channelCount should be 1

The spec says channelCount should be 1.

Also change the default number of output channels to be 1 (was 2).
This reduces complexity of any down-stream nodes so that they will
process mono unless the input to the AnalyserNode has more than one
channel.  Without this, all down-stream nodes will see a stereo input,
doubling processing costs.  This change should not be user-detectable
in any way.

BUG=706610
TEST=constructor/analyser.html
==========

to

==========
AnalyserNode.channelCount should be 1

The spec says channelCount should be 1.

Also change the default number of output channels to be 1 (was 2).
This reduces complexity of any down-stream nodes so that they will
process mono unless the input to the AnalyserNode has more than one
channel.  Without this, all down-stream nodes will see a stereo input,
doubling processing costs.  This change should not be user-detectable
in any way.

BUG=706610
TEST=constructor/analyser.html

Review-Url: https://codereview.chromium.org/2809923004
Cr-Commit-Position: refs/heads/master@{#466787}
Committed:
https://chromium.googlesource.com/chromium/src/+/2dbd3497a559c2227a37a4e108f6...
==========

[commit-bot: I haz the power, 2017-04-24 22:06:52]

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