Fix the output device selection introduced by 163343002

Fix the output device selection introduced by 163343002.

It corrects a mistake which does not assign the output device id to the member of the dispatcher.

NOTRY=true

BUG=344818
TEST=http://localhost:8080/hangout_services_test.html works with device selection.
R=joi@chromium.org

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

[no longer working on chromium, 2014-02-19 15:04:39]

Joi, could you please review this small fix?

Thanks,
SX

[Jói, 2014-02-19 15:09:15]

LGTM

On Wed, Feb 19, 2014 at 3:04 PM,  <xians@chromium.org> wrote:
> Reviewers: Jói,
>
> Message:
>
> Joi, could you please review this small fix?
>
> Thanks,
> SX
>
> Description:
> Fix the output device selection introduced by 163343002.
>
> It corrects a mistake which does not assign the output device id to the
> member
> of the dispatcher.
>
> BUG=344818
> TEST=http://localhost:8080/hangout_services_test.html works with device
> selection.
>
> Please review this at https://codereview.chromium.org/172243003/
>
> SVN Base: http://git.chromium.org/chromium/src.git@master
>
> Affected files (+2, -1 lines):
>   M media/audio/audio_manager_base.cc
>
>
> Index: media/audio/audio_manager_base.cc
> diff --git a/media/audio/audio_manager_base.cc
> b/media/audio/audio_manager_base.cc
> index
>
f9dcafe7fdecc1b160e14e8b36a5e18efb1e2fda..0b192523d4b3f65e624246b3c7b712053288a7da
> 100644
> --- a/media/audio/audio_manager_base.cc
> +++ b/media/audio/audio_manager_base.cc
> @@ -39,7 +39,8 @@ struct AudioManagerBase::DispatcherParams {
>                     const AudioParameters& output,
>                     const std::string& output_device_id)
>        : input_params(input),
> -        output_params(output) {}
> +        output_params(output),
> +        output_device_id(output_device_id) {}
>    ~DispatcherParams() {}
>
>    const AudioParameters input_params;
>
>

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-02-19 15:14:58]

The CQ bit was checked by xians@chromium.org

[commit-bot: I haz the power, 2014-02-19 15:15:03]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xians@chromium.org/172243003/1

[commit-bot: I haz the power, 2014-02-19 15:38:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xians@chromium.org/172243003/1

[no longer working on chromium, 2014-02-19 16:55:04]

Committed patchset #2 manually as r252025 (tree was closed).

[no longer working on chromium, 2014-02-19 16:56:05]

On 2014/02/19 16:55:04, xians1 wrote:
> Committed patchset #2 manually as r252025 (tree was closed).

Note that all the try jobs have been done in Patch Set 1, and no change in Patch
set 2.