tree ce1931bdc8210da9f93c45ab3a72f00a6b0dd789
parent cdbebb086e0d783bab3808da98397f4eb022ae7e
author Jianjun Zhu <jianjun.zhu@intel.com> 1590486197 +0800
committer Commit Bot <commit-bot@chromium.org> 1590493877 +0000

Fix an incorrect use of iterator.

This change uses index instead of iterator to access elements in
`all_targets` to avoid using invalidated iterator after insertion.

MSVC 2019 reports "cannot increment value-initialized deque iterator".
And C++ standard says "an insertion at either end of the deque
invalidates all the iterators to the deque".

Bug: webrtc:11255
Change-Id: I2167bfe875bb0059e81eba334bbd6921e287d6d9
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/176101
Commit-Queue: Karl Wiberg <kwiberg@webrtc.org>
Reviewed-by: Sebastian Jansson <srte@webrtc.org>
Reviewed-by: Karl Wiberg <kwiberg@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31354}