Add variable use_concatenated_impulse_responses to control whether webaudio uses concatenated impul…

Add variable use_concatenated_impulse_responses to control whether webaudio uses concatenated impulse responses.

BUG=227141
This is a prerequisite for https://codereview.chromium.org/14304002/.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=196458

[scottmg, 2013-04-25 16:55:58]

when is use_concatenated_impulse_responses set to 1? the other cl says "all
platforms"?

add 227141 to the BUG= line in the cl description

[Raymond Toy (Google), 2013-04-25 17:01:06]

On 2013/04/25 16:55:58, scottmg wrote:
> when is use_concatenated_impulse_responses set to 1? the other cl says "all
> platforms"?

When the other CL has landed, we can set use_concatenated_impulse_responses to
1, thereby enabling it for all platforms.

> 
> add 227141 to the BUG= line in the cl description

[scottmg, 2013-04-25 17:03:32]

On 2013/04/25 17:01:06, rtoy wrote:
> On 2013/04/25 16:55:58, scottmg wrote:
> > when is use_concatenated_impulse_responses set to 1? the other cl says "all
> > platforms"?
> 
> When the other CL has landed, we can set use_concatenated_impulse_responses to
> 1, thereby enabling it for all platforms.

OK, so it'll be removed again after the other one lands? lgtm in that case.

> 
> > 
> > add 227141 to the BUG= line in the cl description

[Raymond Toy (Google), 2013-04-25 17:10:57]

On 2013/04/25 17:03:32, scottmg wrote:
> On 2013/04/25 17:01:06, rtoy wrote:
> > On 2013/04/25 16:55:58, scottmg wrote:
> > > when is use_concatenated_impulse_responses set to 1? the other cl says
"all
> > > platforms"?
> > 
> > When the other CL has landed, we can set use_concatenated_impulse_responses
to
> > 1, thereby enabling it for all platforms.
> 
> OK, so it'll be removed again after the other one lands? lgtm in that case.

No, it will stay so we can choose whether to use the concatenated version or
not.  This is needed if the impulse responses need to be updated and the
non-concatenated version is much easier to work with.

scottmg mentioned on IRC that this is still ok with him.
> 
> > 
> > > 
> > > add 227141 to the BUG= line in the cl description

[commit-bot: I haz the power, 2013-04-25 17:11:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtoy@google.com/14311015/1

[commit-bot: I haz the power, 2013-04-25 19:35:43]

Change committed as 196458