tree f1b64a49c2c6f38d5c79e7d71e9f4ccaeb27cb70
parent 154fd46c8b296459140475ac751c2ac5f522a500
author Andrey Loskutov <loskutov@gmx.de> 1473421039 +0200
committer Andrey Loskutov <loskutov@gmx.de> 1478458618 -0400

Bug 428417 - ConcurrentModificationException in
AbstractMarkerAnnotationModel

The fDeletedAnnotations field in AbstractMarkerAnnotationModel uses
ArrayList, which is not thread safe and does not allow concurrent
modification and iteration. Introducing new locks around
fDeletedAnnotations seem to be dangerous. Replacing this with
CopyOnWriteArrayList however seem to be a better solution - it does not
add any new locks and should allow safe iteration while other thread
modifies the list. Looking at the usage of fDeletedAnnotations, it
should be OK to iterate on "old" state.

Change-Id: I17fea70fcccf578212e9fdb557a9f0f68aa64dc4
Signed-off-by: Andrey Loskutov <loskutov@gmx.de>
(cherry picked from commit 6540c4c82552458c680d5c01672d0c573db2efd7)