[Android] Fix iteration in GinJavaBridgeDispatcherHost::RemoveHolderAndAdvanceLocked

[Android] Fix iteration in GinJavaBridgeDispatcherHost::RemoveHolderAndAdvanceLocked

Make sure that iterator is always updated to point to the next element.

BUG=484927
Committed: https://crrev.com/cf4a81913bc377473f3bfdf9fed67da3208ca79f
Cr-Commit-Position: refs/heads/master@{#328544}

[mnaganov (inactive), 2015-05-06 01:13:01]

mnaganov@chromium.org changed reviewers:
+ torne@chromium.org

[mnaganov (inactive), 2015-05-06 01:13:02]

*facepalm*

[Torne, 2015-05-06 10:35:41]

I didn't spot it either when I reviewed the original CL, so whoops. LGTM.

[mnaganov (inactive), 2015-05-06 15:17:06]

The CQ bit was checked by mnaganov@chromium.org

[commit-bot: I haz the power, 2015-05-06 15:17:46]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1124003003/1

[commit-bot: I haz the power, 2015-05-06 16:03:47]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-05-06 16:04:37]

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

[sgurun-gerrit only, 2015-05-06 18:55:02]

On 2015/05/06 16:04:37, I haz the power (commit-bot) wrote:
> Patchset 1 (id:??) landed as
> https://crrev.com/cf4a81913bc377473f3bfdf9fed67da3208ca79f
> Cr-Commit-Position: refs/heads/master@{#328544}

would it be possible to write a test for this?

[mnaganov (inactive), 2015-05-06 19:25:34]

On 2015/05/06 18:55:02, sgurun wrote:
> On 2015/05/06 16:04:37, I haz the power (commit-bot) wrote:
> > Patchset 1 (id:??) landed as
> > https://crrev.com/cf4a81913bc377473f3bfdf9fed67da3208ca79f
> > Cr-Commit-Position: refs/heads/master@{#328544}
> 
> would it be possible to write a test for this?

Should be. I was just so much excited about finding and fixing the bug that
totally forgot about the not so much exciting part of writing a regression test.