Correct browser side audio effect state for mediastream requests.

Correct browser side audio effect state for mediastream requests.

The problem was that MediaStreamDispatcher::IsAudioDuckingActive could
return true incorrectly (i.e. when an input device was open but it was
not actually opened with ducking enabled - as specified by the application).

The reason for this is that the audio effect state as known by
MediaStreamDispatcher reflects what the browser side delivers when an
audio track has been opened but the browser side had not taken into
account the stream options of the request.

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

[tommi (sloooow) - chröme, 2014-08-12 14:24:09]

Fix comment

[tommi (sloooow) - chröme, 2014-08-12 14:37:13]

Oops, this was supposed to be a proper review request... sorry about that.

Victoria - can you please take a look?


On Tue, Aug 12, 2014 at 4:24 PM, <tommi@chromium.org> wrote:

> https://codereview.chromium.org/464923002/
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[no longer working on chromium, 2014-08-13 12:50:23]

lgtm

[tommi (sloooow) - chröme, 2014-08-13 13:08:06]

The CQ bit was checked by tommi@chromium.org

[commit-bot: I haz the power, 2014-08-13 13:09:15]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tommi@chromium.org/464923002/20001

[commit-bot: I haz the power, 2014-08-13 14:16:52]

Committed patchset #2 (20001) as 289286