fix: 修复调度问题

xiaocydx · xiaocydx · commit c3dade18882b · 2025-06-12T11:26:27.000+08:00
diff --git a/cxrv/src/main/kotlin/com/xiaocydx/cxrv/internal/Traversal.kt b/cxrv/src/main/kotlin/com/xiaocydx/cxrv/internal/Traversal.kt
@@ -17,6 +17,7 @@
 package com.xiaocydx.cxrv.internal
 
 import android.content.Context
+import android.graphics.Canvas
 import android.view.Choreographer
 import android.view.Choreographer.FrameCallback
 import android.view.KeyEvent
@@ -88,10 +89,13 @@ private class BroadcastProxy(private val view: View) {
             return
         }
         callbacks += callback
-        view.requestLayout()
         broadcastFrame?.scheduleTraversal()
     }
 
+    fun requestLayoutIfNecessary() {
+        if (callbacks.size > 0) view.requestLayout()
+    }
+
     fun removeTraversalCallback(callback: FrameCallback) {
         callbacks -= callback
     }
@@ -116,13 +120,10 @@ private class BroadcastProxy(private val view: View) {
 }
 
 private class BroadcastFrame private constructor(context: Context) : View(context) {
-    private var canUnregisterOnDetached = true
+    private val scheduleTraversalRunner = Runnable { scheduleTraversal() }
     private var proxyList = InlineList<BroadcastProxy>()
-
-    init {
-        setWillNotDraw(true)
-        visibility = INVISIBLE
-    }
+    private var canUnregisterOnDetached = true
+    private var isScheduled = false
 
     fun register(proxy: BroadcastProxy) {
         proxyList += proxy
@@ -133,6 +134,9 @@ private class BroadcastFrame private constructor(context: Context) : View(contex
     }
 
     fun scheduleTraversal() {
+        removeCallbacks(scheduleTraversalRunner)
+        proxyList.reverseAccessEach { it.requestLayoutIfNecessary() }
+
         ensureFirstMeasure()
         requestLayout()
         if (parent !is FrameLayout) {
@@ -141,6 +145,13 @@ private class BroadcastFrame private constructor(context: Context) : View(contex
             // requestApplyInsets()&#26159;&#26367;&#34917;&#26041;&#26696;&#65292;&#23613;&#21487;&#33021;&#35753;BroadcastFrame&#22312;&#20854;&#23427;child&#20043;&#21069;&#25191;&#34892;&#12290;
             requestApplyInsets()
         }
+
+        // &#35843;&#29992;scheduleTraversal()&#26102;&#65292;&#24403;&#21069;&#21487;&#33021;&#22788;&#20110;measure&#12289;layout&#38454;&#27573;&#65292;
+        // &#27492;&#26102;&#35843;&#29992;requestLayout()&#20250;&#34987;parent&#30340;flag&#25318;&#25130;&#65292;&#19981;&#20250;&#26377;&#19979;&#19968;&#24103;&#12290;
+        // &#22240;&#27492;&#35843;&#29992;invalidate()&#65292;&#22312;onDraw()&#21028;&#26029;isScheduled&#26159;&#21542;&#34987;&#37325;&#32622;&#65292;
+        // &#33509;isScheduled&#26410;&#34987;&#37325;&#32622;&#65292;&#21017;&#34920;&#31034;&#34987;&#25318;&#25130;&#65292;&#38656;&#35201;&#37325;&#26032;&#35843;&#24230;&#12290;
+        invalidate()
+        isScheduled = true
     }
 
     override fun dispatchApplyWindowInsets(insets: WindowInsets): WindowInsets {
@@ -153,7 +164,16 @@ private class BroadcastFrame private constructor(context: Context) : View(contex
         setMeasuredDimension(0, 0)
     }
 
+    override fun onDraw(canvas: Canvas) {
+        if (isScheduled) {
+            isScheduled = false
+            // &#21069;&#32622;&#27969;&#31243;&#21487;&#33021;&#20250;&#28155;&#21152;&#21516;&#27493;&#23631;&#38556;&#65292;&#22240;&#27492;&#19981;&#20351;&#29992;post()
+            postOnAnimation(scheduleTraversalRunner)
+        }
+    }
+
     private fun consumeTraversalCallbacks() {
+        isScheduled = false
         proxyList.reverseAccessEach { it.consumeTraversalCallbacks() }
     }
 
@@ -171,6 +191,7 @@ private class BroadcastFrame private constructor(context: Context) : View(contex
     override fun onDetachedFromWindow() {
         super.onDetachedFromWindow()
         if (!canUnregisterOnDetached) return
+        removeCallbacks(scheduleTraversalRunner)
         proxyList.reverseAccessEach { it.unregisterFromBroadcastFrame() }
         proxyList = proxyList.clear()
     }
