Adds stereo support for Java-based input and output audio on Android

Adds stereo support for Java-based input and output audio on Android

BUG=webrtc:6718
Committed: https://crrev.com/779017d989b01b53954636a7237e81dd249e5b84
Cr-Commit-Position: refs/heads/master@{#15104}

[henrika_webrtc, 2016-11-11 15:18:27]

Description was changed from

==========
Adds stereo support for Java-based input and output audio on Android

BUG=
==========

to

==========
Adds stereo support for Java-based input and output audio on Android

BUG=NONE
==========

[henrika_webrtc, 2016-11-16 11:39:14]

Description was changed from

==========
Adds stereo support for Java-based input and output audio on Android

BUG=NONE
==========

to

==========
Adds stereo support for Java-based input and output audio on Android

BUG=webrtc:6718
==========

[henrika_webrtc, 2016-11-16 13:18:50]

henrika@webrtc.org changed reviewers:
+ magjed@webrtc.org

[henrika_webrtc, 2016-11-16 13:18:59]

PTAL

[magjed_webrtc, 2016-11-16 13:42:09]

https://codereview.webrtc.org/2499613002/diff/80001/webrtc/modules/audio_devi...
File webrtc/modules/audio_device/android/audio_track_jni.h (right):

https://codereview.webrtc.org/2499613002/diff/80001/webrtc/modules/audio_devi...
webrtc/modules/audio_device/android/audio_track_jni.h:145: size_t
bytes_per_frame_;
Do we need to store this as a member variable, or can we call
'audio_parameters_.channels() * sizeof(int16_t)' when we need it instead?

[henrika_webrtc, 2016-11-16 13:56:09]

The CQ bit was checked by henrika@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-16 13:56:19]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[henrika_webrtc, 2016-11-16 13:56:38]

Done. Thanks.

PTAL

https://codereview.webrtc.org/2499613002/diff/80001/webrtc/modules/audio_devi...
File webrtc/modules/audio_device/android/audio_track_jni.h (right):

https://codereview.webrtc.org/2499613002/diff/80001/webrtc/modules/audio_devi...
webrtc/modules/audio_device/android/audio_track_jni.h:145: size_t
bytes_per_frame_;
On 2016/11/16 13:42:09, magjed_webrtc wrote:
> Do we need to store this as a member variable, or can we call
> 'audio_parameters_.channels() * sizeof(int16_t)' when we need it instead?

Done.

[magjed_webrtc, 2016-11-16 13:59:22]

lgtm

[commit-bot: I haz the power, 2016-11-16 14:01:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-16 14:01:08]

Dry run: Try jobs failed on following builders:
  mac_baremetal on master.tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/mac_baremetal/builds/16269)

[henrika_webrtc, 2016-11-16 14:02:21]

The CQ bit was checked by henrika@webrtc.org

[commit-bot: I haz the power, 2016-11-16 14:02:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-11-16 14:30:48]

Description was changed from

==========
Adds stereo support for Java-based input and output audio on Android

BUG=webrtc:6718
==========

to

==========
Adds stereo support for Java-based input and output audio on Android

BUG=webrtc:6718
==========

[commit-bot: I haz the power, 2016-11-16 14:30:49]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-11-16 14:30:56]

Description was changed from

==========
Adds stereo support for Java-based input and output audio on Android

BUG=webrtc:6718
==========

to

==========
Adds stereo support for Java-based input and output audio on Android

BUG=webrtc:6718

Committed: https://crrev.com/779017d989b01b53954636a7237e81dd249e5b84
Cr-Commit-Position: refs/heads/master@{#15104}
==========

[commit-bot: I haz the power, 2016-11-16 14:30:56]

Patchset 6 (id:??) landed as
https://crrev.com/779017d989b01b53954636a7237e81dd249e5b84
Cr-Commit-Position: refs/heads/master@{#15104}