Reorder FrameTimingsObserver methods (#55743)

Summary:
Pull Request resolved: #55743

Small refactor — locate public methods first, sort private methods in call order.

Changelog: [Internal]

Reviewed By: rubennorte

Differential Revision: D94366535

fbshipit-source-id: a41f1e6d0b0a50e9733f677b1149ca4b4a03b03e

Author: huntie

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/inspector/FrameTimingsObserver.kt

@@ -38,12 +38,35 @@ internal class FrameTimingsObserver(
 
   @Volatile private var currentWindow: Window? = null
 
-  private val frameMetricsListener =
-      Window.OnFrameMetricsAvailableListener { _, frameMetrics, _dropCount ->
-        val beginTimestamp = frameMetrics.getMetric(FrameMetrics.VSYNC_TIMESTAMP)
-        val endTimestamp = beginTimestamp + frameMetrics.getMetric(FrameMetrics.TOTAL_DURATION)
-        emitFrameTiming(beginTimestamp, endTimestamp)
-      }
+  fun start() {
+    if (!isSupported) {
+      return
+    }
+
+    frameCounter = 0
+    isStarted = true
+
+    // Capture initial screenshot to ensure there's always at least one frame
+    // recorded at the start of tracing, even if no UI changes occur
+    val timestamp = System.nanoTime()
+    emitFrameTiming(timestamp, timestamp)
+
+    currentWindow?.addOnFrameMetricsAvailableListener(frameMetricsListener, handler)
+  }
+
+  fun stop() {
+    if (!isSupported) {
+      return
+    }
+
+    isStarted = false
+
+    currentWindow?.removeOnFrameMetricsAvailableListener(frameMetricsListener)
+    handler.removeCallbacksAndMessages(null)
+
+    bitmapBuffer?.recycle()
+    bitmapBuffer = null
+  }
 
   fun setCurrentWindow(window: Window?) {
     if (!isSupported || currentWindow === window) {
@@ -57,6 +80,32 @@ internal class FrameTimingsObserver(
     }
   }
 
+  private val frameMetricsListener =
+      Window.OnFrameMetricsAvailableListener { _, frameMetrics, _ ->
+        val beginTimestamp = frameMetrics.getMetric(FrameMetrics.VSYNC_TIMESTAMP)
+        val endTimestamp = beginTimestamp + frameMetrics.getMetric(FrameMetrics.TOTAL_DURATION)
+        emitFrameTiming(beginTimestamp, endTimestamp)
+      }
+
+  private fun emitFrameTiming(beginTimestamp: Long, endTimestamp: Long) {
+    val frameId = frameCounter++
+    val threadId = Process.myTid()
+
+    CoroutineScope(Dispatchers.Default).launch {
+      val screenshot = if (screenshotsEnabled) captureScreenshot() else null
+
+      onFrameTimingSequence(
+          FrameTimingSequence(
+              frameId,
+              threadId,
+              beginTimestamp,
+              endTimestamp,
+              screenshot,
+          )
+      )
+    }
+  }
+
   private suspend fun captureScreenshot(): String? = suspendCoroutine { continuation ->
     if (Build.VERSION.SDK_INT < Build.VERSION_CODES.O) {
       continuation.resume(null)
@@ -122,53 +171,4 @@ internal class FrameTimingsObserver(
         handler,
     )
   }
-
-  fun start() {
-    if (!isSupported) {
-      return
-    }
-
-    frameCounter = 0
-    isStarted = true
-
-    // Capture initial screenshot to ensure there's always at least one frame
-    // recorded at the start of tracing, even if no UI changes occur
-    val timestamp = System.nanoTime()
-    emitFrameTiming(timestamp, timestamp)
-
-    currentWindow?.addOnFrameMetricsAvailableListener(frameMetricsListener, handler)
-  }
-
-  private fun emitFrameTiming(beginTimestamp: Long, endTimestamp: Long) {
-    val frameId = frameCounter++
-    val threadId = Process.myTid()
-
-    CoroutineScope(Dispatchers.Default).launch {
-      val screenshot = if (screenshotsEnabled) captureScreenshot() else null
-
-      onFrameTimingSequence(
-          FrameTimingSequence(
-              frameId,
-              threadId,
-              beginTimestamp,
-              endTimestamp,
-              screenshot,
-          )
-      )
-    }
-  }
-
-  fun stop() {
-    if (!isSupported) {
-      return
-    }
-
-    isStarted = false
-
-    currentWindow?.removeOnFrameMetricsAvailableListener(frameMetricsListener)
-    handler.removeCallbacksAndMessages(null)
-
-    bitmapBuffer?.recycle()
-    bitmapBuffer = null
-  }
 }