Redraw tiles once per burst of tiled-marker changes

src/main/java/com/mapconductor/here/marker/HereMarkerController.kt

@@ -22,7 +22,10 @@ import com.mapconductor.core.tileserver.TileServerRegistry
 import com.mapconductor.here.HereActualMarker
 import com.mapconductor.here.HereViewHolder
 import java.util.UUID
+import kotlin.time.Duration.Companion.milliseconds
 import android.os.SystemClock
+import kotlinx.coroutines.Job
+import kotlinx.coroutines.delay
 import kotlinx.coroutines.launch
 import kotlinx.coroutines.sync.withPermit
 
@@ -112,11 +115,12 @@ class HereMarkerController private constructor(
         if (currentFinger == prevFinger) return
 
         semaphore.withPermit {
+            // 件数だけが要る。allEntities() は全件の写しを作るので、ラベルの付け替えのように
+            // 数十本が続けて変わると、そのたびに 14 万件を写すことになる。
             val tilingEnabled =
-                markerTiling.enabled && markerManager.allEntities().size >= markerManager.minMarkerCount
+                markerTiling.enabled && markerManager.getMemoryStats().entityCount >= markerManager.minMarkerCount
             val wantsTiled = tilingEnabled && !state.draggable && state.getAnimation() == null
             val wasTiled = tiledMarkerIds.contains(state.id)
-            val markerIcon = state.icon?.toBitmapIcon() ?: defaultMarkerIcon
 
             if (wantsTiled) {
                 if (!wasTiled) {
@@ -134,14 +138,19 @@ class HereMarkerController private constructor(
                         tiling = true,
                     ),
                 )
-                renderer.onPostProcess()
-                syncTiledOverlay()
+                // タイル担当のままなら、ネイティブのマーカーには何も起きていない。onPostProcess は
+                // 全件を写してネイティブ分を描き直すので（14 万件で 1 回 50ms）、ネイティブから
+                // タイルへ移ったときだけ呼ぶ。
+                if (!wasTiled) renderer.onPostProcess()
+                scheduleTileRefresh()
                 return@withPermit
             }
 
             if (wasTiled) {
                 tiledMarkerIds.remove(state.id)
             }
+            // ネイティブマーカーとして描くときだけ焼き込む。タイル担当なら要らない。
+            val markerIcon = state.icon?.toBitmapIcon() ?: defaultMarkerIcon
 
             val params =
                 object : MarkerOverlayRendererInterface.ChangeParamsInterface<HereActualMarker> {
@@ -172,14 +181,33 @@ class HereMarkerController private constructor(
             renderer.onPostProcess()
 
             if (tiledMarkerIds.isNotEmpty()) {
-                syncTiledOverlay()
+                scheduleTileRefresh()
             } else {
                 removeTileOverlay()
             }
         }
         viewportSwitch.requestReapply()
     }
 
+    private var tileRefreshJob: Job? = null
+
+    /**
+     * 1 本ずつの変更からの描き直しを 1 回にまとめる。
+     *
+     * ラベルの付け替えのように数十本が続けて変わると、1 本ごとにラスターソースを差し替え、
+     * そのたびに地図が見えているタイルを全部取り直していた（MapLibre・Pixel 5a で 45 回、
+     * 描き直したタイル 494 枚、最後のタイルまで 6 秒）。続けて来た変更は
+     * [TILE_REFRESH_COALESCE] の間に受けて、まとめて 1 回だけ差し替える。
+     */
+    private fun scheduleTileRefresh() {
+        if (tileRefreshJob?.isActive == true) return
+        tileRefreshJob =
+            (renderer as HereMarkerRenderer).coroutine.launch {
+                delay(TILE_REFRESH_COALESCE)
+                semaphore.withPermit { syncTiledOverlay() }
+            }
+    }
+
     override suspend fun clear() {
         viewportSwitch.destroy()
         semaphore.withPermit {
@@ -216,6 +244,8 @@ class HereMarkerController private constructor(
 
     override fun destroy() {
         viewportSwitch.destroy()
+        // 待っている描き直しが後から走ると、外したタイル経路をまた登録してしまう。
+        tileRefreshJob?.cancel()
         // Clean up tile server registration
         // Unregister this map's route only. Never stop the server here: it is
         // a process-wide singleton shared by all map controllers and overlay
@@ -328,6 +358,9 @@ class HereMarkerController private constructor(
     }
 
     companion object {
+        /** 続けて来る変更を 1 回の描き直しにまとめる窓。1 フレーム分。 */
+        private val TILE_REFRESH_COALESCE = 16.milliseconds
+
         fun create(
             holder: HereViewHolder,
             markerTiling: MarkerTilingOptions = MarkerTilingOptions.Default,