Ensures that AudioDeviceBuffer::StopPeriodicLogging works as intended

Ensures that AudioDeviceBuffer::StopPeriodicLogging works as intended.

Minor fix to resolve https://bugs.chromium.org/p/webrtc/issues/detail?id=6560&desc=2#c5

BUG=webrtc:6560
NOTRY=TRUE

Committed: https://crrev.com/0b3a6389c054dff7cc9da10a326eedbd4d9f5e61
Cr-Commit-Position: refs/heads/master@{#15036}

[henrika_webrtc, 2016-11-10 14:22:31]

Description was changed from

==========
Ensures that AudioDeviceBuffer::StopPeriodicLogging works as intended

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

to

==========
Ensures that AudioDeviceBuffer::StopPeriodicLogging works as intended

BUG=webrtc:6560
==========

[henrika_webrtc, 2016-11-10 14:22:31]

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

[henrika_webrtc, 2016-11-10 14:22:57]

Description was changed from

==========
Ensures that AudioDeviceBuffer::StopPeriodicLogging works as intended

BUG=webrtc:6560
==========

to

==========
Ensures that AudioDeviceBuffer::StopPeriodicLogging works as intended.

Minor fix to resolve
https://bugs.chromium.org/p/webrtc/issues/detail?id=6560&desc=2#c5

BUG=webrtc:6560
==========

[henrika_webrtc, 2016-11-10 14:23:11]

PTAL

[henrika_webrtc, 2016-11-10 14:54:16]

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

[commit-bot: I haz the power, 2016-11-10 14:54:27]

Dry run: 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-10 15:55:38]

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

[commit-bot: I haz the power, 2016-11-10 15:55:39]

Dry run: This issue passed the CQ dry run.

[kwiberg-webrtc, 2016-11-11 09:20:49]

lgtm

https://codereview.webrtc.org/2496543002/diff/20001/webrtc/modules/audio_devi...
File webrtc/modules/audio_device/audio_device_buffer.cc (right):

https://codereview.webrtc.org/2496543002/diff/20001/webrtc/modules/audio_devi...
webrtc/modules/audio_device/audio_device_buffer.cc:443: return;
Consider eliminating this early return, since it no longer serves a purpose now
that you added another early return on line 450.

[henrika_webrtc, 2016-11-11 10:26:50]

Thanks. Done.

https://codereview.webrtc.org/2496543002/diff/20001/webrtc/modules/audio_devi...
File webrtc/modules/audio_device/audio_device_buffer.cc (right):

https://codereview.webrtc.org/2496543002/diff/20001/webrtc/modules/audio_devi...
webrtc/modules/audio_device/audio_device_buffer.cc:443: return;
On 2016/11/11 09:20:49, kwiberg-webrtc wrote:
> Consider eliminating this early return, since it no longer serves a purpose
now
> that you added another early return on line 450.

Done.

[henrika_webrtc, 2016-11-11 10:27:05]

Description was changed from

==========
Ensures that AudioDeviceBuffer::StopPeriodicLogging works as intended.

Minor fix to resolve
https://bugs.chromium.org/p/webrtc/issues/detail?id=6560&desc=2#c5

BUG=webrtc:6560
==========

to

==========
Ensures that AudioDeviceBuffer::StopPeriodicLogging works as intended.

Minor fix to resolve
https://bugs.chromium.org/p/webrtc/issues/detail?id=6560&desc=2#c5

BUG=webrtc:6560
NOTRY=TRUE
==========

[henrika_webrtc, 2016-11-11 10:27:09]

The CQ bit was checked by henrika@webrtc.org

[henrika_webrtc, 2016-11-11 10:27:09]

The patchset sent to the CQ was uploaded after l-g-t-m from kwiberg@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2496543002/#ps40001 (title:
"Feedback from kwiberg@")

[commit-bot: I haz the power, 2016-11-11 10:27:21]

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-11 10:28:52]

Description was changed from

==========
Ensures that AudioDeviceBuffer::StopPeriodicLogging works as intended.

Minor fix to resolve
https://bugs.chromium.org/p/webrtc/issues/detail?id=6560&desc=2#c5

BUG=webrtc:6560
NOTRY=TRUE
==========

to

==========
Ensures that AudioDeviceBuffer::StopPeriodicLogging works as intended.

Minor fix to resolve
https://bugs.chromium.org/p/webrtc/issues/detail?id=6560&desc=2#c5

BUG=webrtc:6560
NOTRY=TRUE
==========

[commit-bot: I haz the power, 2016-11-11 10:28:53]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-11 10:29:04]

Description was changed from

==========
Ensures that AudioDeviceBuffer::StopPeriodicLogging works as intended.

Minor fix to resolve
https://bugs.chromium.org/p/webrtc/issues/detail?id=6560&desc=2#c5

BUG=webrtc:6560
NOTRY=TRUE
==========

to

==========
Ensures that AudioDeviceBuffer::StopPeriodicLogging works as intended.

Minor fix to resolve
https://bugs.chromium.org/p/webrtc/issues/detail?id=6560&desc=2#c5

BUG=webrtc:6560
NOTRY=TRUE

Committed: https://crrev.com/0b3a6389c054dff7cc9da10a326eedbd4d9f5e61
Cr-Commit-Position: refs/heads/master@{#15036}
==========

[commit-bot: I haz the power, 2016-11-11 10:29:05]

Patchset 3 (id:??) landed as
https://crrev.com/0b3a6389c054dff7cc9da10a326eedbd4d9f5e61
Cr-Commit-Position: refs/heads/master@{#15036}