CHECK for adding duplicate converter inputs.

CHECK for adding duplicate converter inputs.

CHECK to see if we're getting into weird states due to inputs
being added twice.

BUG=233026
TEST=none

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

[scherkus (not reviewing), 2013-04-24 22:06:41]

lgtm w/ nit

https://codereview.chromium.org/14083011/diff/1/media/base/audio_converter.cc
File media/base/audio_converter.cc (right):

https://codereview.chromium.org/14083011/diff/1/media/base/audio_converter.cc...
media/base/audio_converter.cc:91: CHECK(std::find(transform_inputs_.begin(),
transform_inputs_.end(), input) ==
if this is a speculative CHECK() can you add TODO + reference bug?

[DaleCurtis, 2013-04-24 22:13:59]

https://codereview.chromium.org/14083011/diff/1/media/base/audio_converter.cc
File media/base/audio_converter.cc (right):

https://codereview.chromium.org/14083011/diff/1/media/base/audio_converter.cc...
media/base/audio_converter.cc:91: CHECK(std::find(transform_inputs_.begin(),
transform_inputs_.end(), input) ==
On 2013/04/24 22:06:42, scherkus wrote:
> if this is a speculative CHECK() can you add TODO + reference bug?

Done.

[commit-bot: I haz the power, 2013-04-24 22:14:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dalecurtis@chromium.org/14083011/5001

[commit-bot: I haz the power, 2013-04-25 00:24:51]

Change committed as 196271