MediaStreamAudioDestination channelCountMode is "explicit"

MediaStreamAudioDestination channelCountMode is "explicit"

The channelCountMode was "max", but it should be "explicit", per the
WebAudio spec.

BUG=715128
TEST=constructor/mediastreamaudiodestination.html

Review-Url: https://codereview.chromium.org/2842213002
Cr-Commit-Position: refs/heads/master@{#472855}
Committed: https://chromium.googlesource.com/chromium/src/+/0ef534a188b3a115411b8b351f2e6e4070bba2e6

[Raymond Toy, 2017-05-08 21:22:50]

Description was changed from

==========
MediaStreamAudioDestination channelCountMode is "explicit"

The channelCountMode was "max", but it should be "explicit", per the
WebAudio spec.

BUG=715128
TEST=
==========

to

==========
MediaStreamAudioDestination channelCountMode is "explicit"

The channelCountMode was "max", but it should be "explicit", per the
WebAudio spec.

BUG=715128
TEST=constructor/mediastreamaudiodestination.html
==========

[Raymond Toy, 2017-05-08 21:23:03]

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

[Raymond Toy, 2017-05-08 21:23:04]

PTAL

[hongchan, 2017-05-10 15:20:05]

lgtm

[Raymond Toy, 2017-05-10 15:59:11]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-05-10 15:59:53]

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-05-10 17:51:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-10 17:51:23]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Raymond Toy, 2017-05-10 19:43:02]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-05-10 19:44:33]

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-05-10 21:52:21]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-10 21:52:21]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Raymond Toy, 2017-05-11 14:40:59]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-05-11 14:41:32]

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-05-11 16:31:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-11 16:31:13]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Raymond Toy, 2017-05-11 17:28:12]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-05-11 17:29:19]

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-05-11 19:24:42]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-11 19:24:43]

Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)

[Raymond Toy, 2017-05-12 15:22:07]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-05-12 15:22:54]

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-05-12 18:10:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-12 18:10:15]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Raymond Toy, 2017-05-12 19:43:46]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-05-12 19:44:49]

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-05-12 19:58:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-12 19:58:55]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Raymond Toy, 2017-05-15 16:00:26]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-05-15 16:00:49]

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-05-15 19:18:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-15 19:18:54]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Raymond Toy, 2017-05-16 14:39:05]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-05-16 14:39:33]

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-05-16 17:46:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-16 17:46:23]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Raymond Toy, 2017-05-16 20:18:27]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-05-16 20:19:55]

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-05-16 23:25:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-16 23:25:19]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Raymond Toy, 2017-05-17 15:32:31]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-05-17 15:33:09]

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-05-17 17:15:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-17 17:15:27]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Raymond Toy, 2017-05-17 19:24:37]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-05-17 19:26:26]

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-05-17 21:37:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-17 21:37:51]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Raymond Toy, 2017-05-18 15:14:03]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2017-05-18 15:14:47]

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-05-18 17:17:28]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1495120443248180,
"parent_rev": "9102badab31062609acf65f4ddf9c3a332e85705", "commit_rev":
"0ef534a188b3a115411b8b351f2e6e4070bba2e6"}

[commit-bot: I haz the power, 2017-05-18 17:17:43]

Description was changed from

==========
MediaStreamAudioDestination channelCountMode is "explicit"

The channelCountMode was "max", but it should be "explicit", per the
WebAudio spec.

BUG=715128
TEST=constructor/mediastreamaudiodestination.html
==========

to

==========
MediaStreamAudioDestination channelCountMode is "explicit"

The channelCountMode was "max", but it should be "explicit", per the
WebAudio spec.

BUG=715128
TEST=constructor/mediastreamaudiodestination.html

Review-Url: https://codereview.chromium.org/2842213002
Cr-Commit-Position: refs/heads/master@{#472855}
Committed:
https://chromium.googlesource.com/chromium/src/+/0ef534a188b3a115411b8b351f2e...
==========

[commit-bot: I haz the power, 2017-05-18 17:17:46]

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