cras_audio_handler add SetOutputMono method

cras_audio_handler add SetOutputMono method

BUG=404701
TEST=cras_audio_handler_unittest.cc

Committed: https://crrev.com/745246282c486ab7a03e40def351a78b7f3958c0
Cr-Commit-Position: refs/heads/master@{#384173}

[Qiang(Joe) Xu, 2016-03-30 22:09:36]

Description was changed from

==========
cras_audio_handler add SetOutputMono method

BUG=404701
TEST=cras_audio_handler_unittest.cc
==========

to

==========
cras_audio_handler add SetOutputMono method

BUG=404701
TEST=cras_audio_handler_unittest.cc
==========

[Qiang(Joe) Xu, 2016-03-30 22:09:36]

warx@chromium.org changed reviewers:
+ jennyz@chromium.org

[Qiang(Joe) Xu, 2016-03-30 22:11:38]

based on the revision:
https://chromium.googlesource.com/chromium/src.git/+/a377c049175591928ea1507e...

[jennyz, 2016-03-30 23:39:26]

LGTM with a nit.

https://codereview.chromium.org/1845733002/diff/1/chromeos/audio/cras_audio_h...
File chromeos/audio/cras_audio_handler.h (right):

https://codereview.chromium.org/1845733002/diff/1/chromeos/audio/cras_audio_h...
chromeos/audio/cras_audio_handler.h:421: int32_t channels_;
How about naming it output_channels_?

[Qiang(Joe) Xu, 2016-03-30 23:56:09]

On 2016/03/30 23:39:26, jennyz wrote:
> LGTM with a nit.
> 
>
https://codereview.chromium.org/1845733002/diff/1/chromeos/audio/cras_audio_h...
> File chromeos/audio/cras_audio_handler.h (right):
> 
>
https://codereview.chromium.org/1845733002/diff/1/chromeos/audio/cras_audio_h...
> chromeos/audio/cras_audio_handler.h:421: int32_t channels_;
> How about naming it output_channels_?

I agree. Done.

[Qiang(Joe) Xu, 2016-03-30 23:56:24]

The CQ bit was checked by warx@chromium.org

[Qiang(Joe) Xu, 2016-03-30 23:56:25]

The patchset sent to the CQ was uploaded after l-g-t-m from jennyz@chromium.org
Link to the patchset: https://codereview.chromium.org/1845733002/#ps20001
(title: "channels_ -> output_channels_")

[commit-bot: I haz the power, 2016-03-30 23:56:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1845733002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1845733002/20001

[commit-bot: I haz the power, 2016-03-31 01:07:35]

Description was changed from

==========
cras_audio_handler add SetOutputMono method

BUG=404701
TEST=cras_audio_handler_unittest.cc
==========

to

==========
cras_audio_handler add SetOutputMono method

BUG=404701
TEST=cras_audio_handler_unittest.cc
==========

[commit-bot: I haz the power, 2016-03-31 01:07:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-31 01:11:06]

Description was changed from

==========
cras_audio_handler add SetOutputMono method

BUG=404701
TEST=cras_audio_handler_unittest.cc
==========

to

==========
cras_audio_handler add SetOutputMono method

BUG=404701
TEST=cras_audio_handler_unittest.cc

Committed: https://crrev.com/745246282c486ab7a03e40def351a78b7f3958c0
Cr-Commit-Position: refs/heads/master@{#384173}
==========

[commit-bot: I haz the power, 2016-03-31 01:11:07]

Patchset 2 (id:??) landed as
https://crrev.com/745246282c486ab7a03e40def351a78b7f3958c0
Cr-Commit-Position: refs/heads/master@{#384173}