Improves resolution when logging rate in the ADB class

Improves resolution when logging rate in the ADB class.

Trivial patch which fixes an issue where logged rate estimates could be
invalid. E.g. on iOS, two successive timer interrupts can be ~10.5 seconds
and not exactly 10.0 (which is usually the case on Android). In those
cases we could log a rate estimate of e.g. ~51000Hz instead of ~48000Hz.

This CL fixes that.

BUG=NONE

Committed: https://crrev.com/a6d26ec6a2a6eccb35b3456010956bf4f53a2659
Cr-Commit-Position: refs/heads/master@{#14305}

[henrika_webrtc, 2016-09-19 14:43:21]

Description was changed from

==========
Improves resolution when logging rate in the ADB class

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

to

==========
Improves resolution when logging rate in the ADB class.

Trivial patch which fixes an issue where logged rate estimates could be
invalid. E.g. on iOS, two successive timer interrupts can be ~10.5 seconds
and not exactly 10.0 (which is usually the case on Android). In those
cases we could log a rate estimate of e.g. ~51000Hz instead of ~48000Hz.

This CL fixes that.

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

[henrika_webrtc, 2016-09-19 14:43:21]

henrika@webrtc.org changed reviewers:
+ henrik.lundin@webrtc.org

[henrika_webrtc, 2016-09-19 14:44:00]

Henrik, PTAL

[hlundin-webrtc, 2016-09-20 07:30:25]

LGTM with one nit.

https://codereview.webrtc.org/2350103002/diff/1/webrtc/modules/audio_device/a...
File webrtc/modules/audio_device/audio_device_buffer.cc (right):

https://codereview.webrtc.org/2350103002/diff/1/webrtc/modules/audio_device/a...
webrtc/modules/audio_device/audio_device_buffer.cc:401: if (++num_stat_reports_
> 1) {
You should add && time_since_last > 0 to the if statement, to avoid div-by-zero.
(Probably not very likely in reality, but at the devil lives on the outskirts of
probability.)

[henrika_webrtc, 2016-09-20 09:05:33]

https://codereview.webrtc.org/2350103002/diff/1/webrtc/modules/audio_device/a...
File webrtc/modules/audio_device/audio_device_buffer.cc (right):

https://codereview.webrtc.org/2350103002/diff/1/webrtc/modules/audio_device/a...
webrtc/modules/audio_device/audio_device_buffer.cc:401: if (++num_stat_reports_
> 1) {
Thanks!

[henrika_webrtc, 2016-09-20 09:05:56]

The CQ bit was checked by henrika@webrtc.org

[henrika_webrtc, 2016-09-20 09:05:56]

The patchset sent to the CQ was uploaded after l-g-t-m from
henrik.lundin@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2350103002/#ps20001 (title:
"Feedback from hlundin@")

[commit-bot: I haz the power, 2016-09-20 09:05:59]

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-09-20 11:44:07]

Description was changed from

==========
Improves resolution when logging rate in the ADB class.

Trivial patch which fixes an issue where logged rate estimates could be
invalid. E.g. on iOS, two successive timer interrupts can be ~10.5 seconds
and not exactly 10.0 (which is usually the case on Android). In those
cases we could log a rate estimate of e.g. ~51000Hz instead of ~48000Hz.

This CL fixes that.

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

to

==========
Improves resolution when logging rate in the ADB class.

Trivial patch which fixes an issue where logged rate estimates could be
invalid. E.g. on iOS, two successive timer interrupts can be ~10.5 seconds
and not exactly 10.0 (which is usually the case on Android). In those
cases we could log a rate estimate of e.g. ~51000Hz instead of ~48000Hz.

This CL fixes that.

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

[commit-bot: I haz the power, 2016-09-20 11:44:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-20 11:44:19]

Description was changed from

==========
Improves resolution when logging rate in the ADB class.

Trivial patch which fixes an issue where logged rate estimates could be
invalid. E.g. on iOS, two successive timer interrupts can be ~10.5 seconds
and not exactly 10.0 (which is usually the case on Android). In those
cases we could log a rate estimate of e.g. ~51000Hz instead of ~48000Hz.

This CL fixes that.

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

to

==========
Improves resolution when logging rate in the ADB class.

Trivial patch which fixes an issue where logged rate estimates could be
invalid. E.g. on iOS, two successive timer interrupts can be ~10.5 seconds
and not exactly 10.0 (which is usually the case on Android). In those
cases we could log a rate estimate of e.g. ~51000Hz instead of ~48000Hz.

This CL fixes that.

BUG=NONE

Committed: https://crrev.com/a6d26ec6a2a6eccb35b3456010956bf4f53a2659
Cr-Commit-Position: refs/heads/master@{#14305}
==========

[commit-bot: I haz the power, 2016-09-20 11:44:20]

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