commit 8fa292ae5b5d935602f2261492ceaf0a6f9eac88
author Austin Wang <wangaustin@google.com> Thu Mar 21 16:35:29 2024 -0700
committer Austin Wang <wangaustin@google.com> Thu Mar 21 23:46:55 2024 +0000
tree a3d2c5ca08ec96c0fb77f8fc75b5c2a4d1b9696f
parent 31445e3f6cf6dbd5d1375c6cd7b6ae9c3e41c599

Fix ConcurrentModificationException when time changed

Flag: NA
Bug: 322409406
Test: on preview wait for time tick
Change-Id: Iad1c97f7537551cc6b9c968257258055db37717f

src/com/android/customization/picker/clock/ui/view/ClockViewFactoryImpl.kt

diff --git a/src/com/android/customization/picker/clock/ui/view/ClockViewFactoryImpl.kt b/src/com/android/customization/picker/clock/ui/view/ClockViewFactoryImpl.kt
index cf9e247..0d603f5 100644
--- a/src/com/android/customization/picker/clock/ui/view/ClockViewFactoryImpl.kt
+++ b/src/com/android/customization/picker/clock/ui/view/ClockViewFactoryImpl.kt
@@ -46,7 +46,7 @@
 ) : ClockViewFactory {
     private val resources = appContext.resources
     private val timeTickListeners: ConcurrentHashMap<Int, TimeTicker> = ConcurrentHashMap()
-    private val clockControllers: HashMap<String, ClockController> = HashMap()
+    private val clockControllers: ConcurrentHashMap<String, ClockController> = ConcurrentHashMap()
     private val smallClockFrames: HashMap<String, FrameLayout> = HashMap()
 
     override fun getController(clockId: String): ClockController {