Fix: Corrected loop variable in DropFramesFrameScheduler

aperson30 · aperson30 · commit d3051329f23f · 2026-02-07T16:16:24.000-08:00
diff --git a/animated-drawable/src/main/java/com/facebook/fresco/animation/frame/DropFramesFrameScheduler.kt b/animated-drawable/src/main/java/com/facebook/fresco/animation/frame/DropFramesFrameScheduler.kt
@@ -46,7 +46,7 @@ class DropFramesFrameScheduler(private val animationInformation: AnimationInform
   override fun getTargetRenderTimeMs(frameNumber: Int): Long {
     var targetRenderTimeMs = 0L
     for (i in 0 until frameNumber) {
-      targetRenderTimeMs += animationInformation.getFrameDurationMs(frameNumber).toLong()
+      targetRenderTimeMs += animationInformation.getFrameDurationMs(i).toLong()
     }
     return targetRenderTimeMs
   }

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ class DropFramesFrameScheduler(private val animationInformation: AnimationInform`
`46`	`46`	`override fun getTargetRenderTimeMs(frameNumber: Int): Long {`
`47`	`47`	`var targetRenderTimeMs = 0L`
`48`	`48`	`for (i in 0 until frameNumber) {`
`49`		`- targetRenderTimeMs += animationInformation.getFrameDurationMs(frameNumber).toLong()`
	`49`	`+ targetRenderTimeMs += animationInformation.getFrameDurationMs(i).toLong()`
`50`	`50`	`}`
`51`	`51`	`return targetRenderTimeMs`
`52`	`52`	`}`