Reset indexer upon initialization in AudioLoop.

Reset indexer upon initialization in AudioLoop.

The array is reset in Init() but not the indexer. This makes the start point undefined after Init() for re-initializing an AudioLoop. This can be fixed.

BUG=

Committed: https://crrev.com/58e08cbea85295ca7aca293c31117994b558e515
Cr-Commit-Position: refs/heads/master@{#11739}

[minyue-webrtc, 2016-02-24 09:30:36]

Description was changed from

==========
reset next_index_

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

to

==========
Reset indexer upon initialization in AudioLoop.

The array is reset in Init() but not the indexer. This makes the start point
undefined after Init() for re-initializing an AudioLoop. This can be fixed.

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

[minyue-webrtc, 2016-02-24 09:30:36]

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

[minyue-webrtc, 2016-02-24 09:30:58]

Henrik,

Here is a small CL for you to review. Thanks!

[minyue-webrtc, 2016-02-24 09:53:55]

Patchset #2 (id:20001) has been deleted

[minyue-webrtc, 2016-02-24 09:59:31]

minyue@webrtc.org changed reviewers:
+ flim@webrtc.org

[minyue-webrtc, 2016-02-24 09:59:31]

+flim

The fixing triggers an earlier error. After Opus 1.1.2 update, the check on a
unittest was relaxed, since check was performed before Opus converges and value
to check depends on the signal. With the fix, the signal becomes different and
the value to check has to change. After all, it is better to check the a tighter
value after Opus converges.

[hlundin-webrtc, 2016-02-24 10:41:26]

Nice catch. LGTM.

[flim-webrtc, 2016-02-24 11:43:46]

lgtm

[minyue-webrtc, 2016-02-24 11:44:17]

The CQ bit was checked by minyue@webrtc.org

[commit-bot: I haz the power, 2016-02-24 11:44:23]

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

[commit-bot: I haz the power, 2016-02-24 11:49:21]

Description was changed from

==========
Reset indexer upon initialization in AudioLoop.

The array is reset in Init() but not the indexer. This makes the start point
undefined after Init() for re-initializing an AudioLoop. This can be fixed.

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

to

==========
Reset indexer upon initialization in AudioLoop.

The array is reset in Init() but not the indexer. This makes the start point
undefined after Init() for re-initializing an AudioLoop. This can be fixed.

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

[commit-bot: I haz the power, 2016-02-24 11:49:22]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-02-24 11:49:30]

Description was changed from

==========
Reset indexer upon initialization in AudioLoop.

The array is reset in Init() but not the indexer. This makes the start point
undefined after Init() for re-initializing an AudioLoop. This can be fixed.

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

to

==========
Reset indexer upon initialization in AudioLoop.

The array is reset in Init() but not the indexer. This makes the start point
undefined after Init() for re-initializing an AudioLoop. This can be fixed.

BUG=

Committed: https://crrev.com/58e08cbea85295ca7aca293c31117994b558e515
Cr-Commit-Position: refs/heads/master@{#11739}
==========

[commit-bot: I haz the power, 2016-02-24 11:49:30]

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