Web Audio: Don't create unnecessary DelayNode object.

Web Audio: Don't create unnecessary DelayNode object.

We can check maxDelayTime argument in the factory to avoid unnecessary DelayNode
creation.

This CL doesn't change user-visible behavior.

BUG=

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=192303

[tkent, 2015-03-20 08:38:22]

tkent@chromium.org changed reviewers:
+ oilpan-reviews@chromium.org, rtoy@chromium.org

[tkent, 2015-03-20 08:38:23]

Please review this.

[haraken, 2015-03-20 13:42:01]

haraken@chromium.org changed reviewers:
+ haraken@chromium.org

[haraken, 2015-03-20 13:42:02]

LGTM

https://codereview.chromium.org/1026663002/diff/1/Source/modules/webaudio/Del...
File Source/modules/webaudio/DelayNode.cpp (right):

https://codereview.chromium.org/1026663002/diff/1/Source/modules/webaudio/Del...
Source/modules/webaudio/DelayNode.cpp:39: :
AudioBasicProcessorNode(NodeTypeDelay, context, sampleRate)

  , m_processor(new DelayProcessor(context, sampleRate, 1, maxDelayTime))

might be a bit better.

[Raymond Toy, 2015-03-20 15:24:04]

lgtm

[tkent, 2015-03-21 04:23:56]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2015-03-21 04:24:19]

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

[commit-bot: I haz the power, 2015-03-21 04:27:38]

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