AudioBufferSourceOptions.buffer should be nullable.

AudioBufferSourceOptions.buffer should be nullable.

It's valid to set the buffer to null in the constructor.

Also added a test for ConvolverOptions which has the same issue, which
was updated in https://codereview.chromium.org/2352463002/

Spec issue: https://github.com/WebAudio/web-audio-api/issues/980

BUG=648283, 626449, 647693
TEST=constructor/audiobuffersource.html, constructor/convolver.html

Committed: https://crrev.com/2582f835f9351586c831e87b0cf69318cf6c971e
Cr-Commit-Position: refs/heads/master@{#419830}

[Raymond Toy, 2016-09-19 16:58:08]

The CQ bit was checked by rtoy@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-19 16:58:27]

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

[Raymond Toy, 2016-09-19 16:58:35]

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

[Raymond Toy, 2016-09-19 16:58:36]

PTAL.

[hongchan, 2016-09-19 17:18:15]

lgtm

[Raymond Toy, 2016-09-19 17:30:32]

Description was changed from

==========
AudioBufferSourceOptions.buffer should be nullable.

It's valid to set the buffer to null in the constructor.

Also added a test for ConvolverOptions which has the same issue, which
was updated in https://codereview.chromium.org/2352463002/

BUG=648283, 626449, 647693
TEST=constructor/audiobuffersource.html, constructor/convolver.html
==========

to

==========
AudioBufferSourceOptions.buffer should be nullable.

It's valid to set the buffer to null in the constructor.

Also added a test for ConvolverOptions which has the same issue, which
was updated in https://codereview.chromium.org/2352463002/

Spec issue: https://github.com/WebAudio/web-audio-api/issues/980


BUG=648283, 626449, 647693
TEST=constructor/audiobuffersource.html, constructor/convolver.html
==========

[Raymond Toy, 2016-09-19 17:31:48]

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

[Raymond Toy, 2016-09-19 17:31:48]

tkent:  PTAL as this really trivial web-exposed change.  It is valid to call
both constructors with {buffer: null}; no error should be thrown.  (This didn't
happen until just a few days ago.)

[commit-bot: I haz the power, 2016-09-19 20:08:10]

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

[commit-bot: I haz the power, 2016-09-19 20:08:11]

Dry run: 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_...)

[tkent, 2016-09-20 00:36:11]

lgtm

[Raymond Toy, 2016-09-20 16:45:53]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2016-09-20 16:46:13]

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, 2016-09-20 18:53:01]

Description was changed from

==========
AudioBufferSourceOptions.buffer should be nullable.

It's valid to set the buffer to null in the constructor.

Also added a test for ConvolverOptions which has the same issue, which
was updated in https://codereview.chromium.org/2352463002/

Spec issue: https://github.com/WebAudio/web-audio-api/issues/980


BUG=648283, 626449, 647693
TEST=constructor/audiobuffersource.html, constructor/convolver.html
==========

to

==========
AudioBufferSourceOptions.buffer should be nullable.

It's valid to set the buffer to null in the constructor.

Also added a test for ConvolverOptions which has the same issue, which
was updated in https://codereview.chromium.org/2352463002/

Spec issue: https://github.com/WebAudio/web-audio-api/issues/980


BUG=648283, 626449, 647693
TEST=constructor/audiobuffersource.html, constructor/convolver.html
==========

[commit-bot: I haz the power, 2016-09-20 18:53:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-20 18:56:13]

Description was changed from

==========
AudioBufferSourceOptions.buffer should be nullable.

It's valid to set the buffer to null in the constructor.

Also added a test for ConvolverOptions which has the same issue, which
was updated in https://codereview.chromium.org/2352463002/

Spec issue: https://github.com/WebAudio/web-audio-api/issues/980


BUG=648283, 626449, 647693
TEST=constructor/audiobuffersource.html, constructor/convolver.html
==========

to

==========
AudioBufferSourceOptions.buffer should be nullable.

It's valid to set the buffer to null in the constructor.

Also added a test for ConvolverOptions which has the same issue, which
was updated in https://codereview.chromium.org/2352463002/

Spec issue: https://github.com/WebAudio/web-audio-api/issues/980

BUG=648283, 626449, 647693
TEST=constructor/audiobuffersource.html, constructor/convolver.html

Committed: https://crrev.com/2582f835f9351586c831e87b0cf69318cf6c971e
Cr-Commit-Position: refs/heads/master@{#419830}
==========

[commit-bot: I haz the power, 2016-09-20 18:56:14]

Patchset 1 (id:??) landed as
https://crrev.com/2582f835f9351586c831e87b0cf69318cf6c971e
Cr-Commit-Position: refs/heads/master@{#419830}