Remove Doppler effect from PannerNode

Remove Doppler effect from PannerNode

The W3C audio group has decided to remove the Doppler effect.  In
addition, the current PannerNode will eventually be deprecated in
favor of new panner nodes (StereoPannerNode and future ones).
(See https://github.com/WebAudio/web-audio-api/issues/372)

The doppler effect was never done in a very general way and was only
applied to AudioBufferSourceNodes, not Oscillators, or MediaStreams.
(The effect was done by modifying the playback rate of the source
node.)  Multiple panner nodes also had strange effects if the velocity
was different in each panner.

This is not a complete removal of the code, but only the part that
causes the source nodes to be modified with a new playback rate based
on the doppler frequency of the panner.  The remaining doppler code
will be removed when the PannerNode is deprecated and finally removed.  Bug 439644 tracks this.

BUG=433798, 439644
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=186738

[Raymond Toy, 2014-12-08 20:43:40]

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

[Raymond Toy, 2014-12-08 20:43:41]

PTAL.

[Ken Russell (switch to Gerrit), 2014-12-08 21:11:18]

hoch@, could you please do the first round review of this CL? Thanks.

[hongchan, 2014-12-08 21:38:11]

On 2014/12/08 21:11:18, Ken Russell wrote:
> hoch@, could you please do the first round review of this CL? Thanks.

As the issue description suggested, this is the minimum change while we can make
the implementation compatible with the current Web Audio API spec. LGTM.

[Ken Russell (switch to Gerrit), 2014-12-08 23:30:08]

On 2014/12/08 21:38:11, hoch wrote:
> On 2014/12/08 21:11:18, Ken Russell wrote:
> > hoch@, could you please do the first round review of this CL? Thanks.
> 
> As the issue description suggested, this is the minimum change while we can
make
> the implementation compatible with the current Web Audio API spec. LGTM.

Thanks. LGTM from my standpoint. Please double-check the CL description before
committing. There's at least one incomplete thought ("(See)").

[Raymond Toy, 2014-12-08 23:36:07]

On 2014/12/08 23:30:08, Ken Russell wrote:
> On 2014/12/08 21:38:11, hoch wrote:
> > On 2014/12/08 21:11:18, Ken Russell wrote:
> > > hoch@, could you please do the first round review of this CL? Thanks.
> > 
> > As the issue description suggested, this is the minimum change while we can
> make
> > the implementation compatible with the current Web Audio API spec. LGTM.
> 
> Thanks. LGTM from my standpoint. Please double-check the CL description before
> committing. There's at least one incomplete thought ("(See)").

Thanks for catching the CL description error.  Updated with appropriate
reference.

[Raymond Toy, 2014-12-08 23:36:16]

The CQ bit was checked by rtoy@chromium.org

[commit-bot: I haz the power, 2014-12-08 23:36:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/783273002/1

[commit-bot: I haz the power, 2014-12-08 23:54:04]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=186738