fix possible gl render ANR 2

pedroSG94 · pedroSG94 · commit 1cd54c21294b · 2026-03-30T23:46:07.000+02:00
diff --git a/library/src/main/java/com/pedro/library/view/OpenGlView.kt b/library/src/main/java/com/pedro/library/view/OpenGlView.kt
@@ -278,9 +278,11 @@ open class OpenGlView : SurfaceView, GlInterface, OnFrameAvailableListener, Surf
     }
 
     override fun addMediaCodecSurface(surface: Surface) {
-      if (surfaceManager.isReady) {
-        surfaceManagerEncoder.release()
-        surfaceManagerEncoder.eglSetup(surface, surfaceManager)
+      executor?.submit {
+        if (surfaceManager.isReady) {
+          surfaceManagerEncoder.release()
+          surfaceManagerEncoder.eglSetup(surface, surfaceManager)
+        }
       }
     }
 
@@ -291,9 +293,11 @@ open class OpenGlView : SurfaceView, GlInterface, OnFrameAvailableListener, Surf
     }
 
     override fun addMediaCodecRecordSurface(surface: Surface) {
-      if (surfaceManager.isReady) {
-        surfaceManagerEncoderRecord.release()
-        surfaceManagerEncoderRecord.eglSetup(surface, surfaceManager)
+      executor?.submit {
+        if (surfaceManager.isReady) {
+          surfaceManagerEncoderRecord.release()
+          surfaceManagerEncoderRecord.eglSetup(surface, surfaceManager)
+        }
       }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -278,9 +278,11 @@ open class OpenGlView : SurfaceView, GlInterface, OnFrameAvailableListener, Surf`
`278`	`278`	`}`
`279`	`279`
`280`	`280`	`override fun addMediaCodecSurface(surface: Surface) {`
`281`		`- if (surfaceManager.isReady) {`
`282`		`- surfaceManagerEncoder.release()`
`283`		`- surfaceManagerEncoder.eglSetup(surface, surfaceManager)`
	`281`	`+ executor?.submit {`
	`282`	`+ if (surfaceManager.isReady) {`
	`283`	`+ surfaceManagerEncoder.release()`
	`284`	`+ surfaceManagerEncoder.eglSetup(surface, surfaceManager)`
	`285`	`+ }`
`284`	`286`	`}`
`285`	`287`	`}`
`286`	`288`
`@@ -291,9 +293,11 @@ open class OpenGlView : SurfaceView, GlInterface, OnFrameAvailableListener, Surf`
`291`	`293`	`}`
`292`	`294`
`293`	`295`	`override fun addMediaCodecRecordSurface(surface: Surface) {`
`294`		`- if (surfaceManager.isReady) {`
`295`		`- surfaceManagerEncoderRecord.release()`
`296`		`- surfaceManagerEncoderRecord.eglSetup(surface, surfaceManager)`
	`296`	`+ executor?.submit {`
	`297`	`+ if (surfaceManager.isReady) {`
	`298`	`+ surfaceManagerEncoderRecord.release()`
	`299`	`+ surfaceManagerEncoderRecord.eglSetup(surface, surfaceManager)`
	`300`	`+ }`
`297`	`301`	`}`
`298`	`302`	`}`
`299`	`303`