commit 518e181ab56094c5856e924be79b40115bb80436
author Linus Nilsson <lnilsson@webrtc.org> Thu Nov 23 13:15:01 2023
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Nov 23 14:56:31 2023
tree 9e4e5124bcedd357537a2d4bd4ab6c97317c3277
parent 0322493aede94db8861b468fc5d54b11e44098ea

Trace render window state using a counter.

This makes it show up as an individual track in perfetto
allowing rendering to be more easily inspected.
https://screenshot.googleplex.com/3eyicjpxdKaG3oE

Bug: b/217863437
Change-Id: I2710a8709141fda50ec613390161ab9c6526b931
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/328381
Auto-Submit: Linus Nilsson <lnilsson@webrtc.org>
Commit-Queue: Linus Nilsson <lnilsson@webrtc.org>
Reviewed-by: Zoé Lepaul <xalep@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#41222}

sdk/android/api/org/webrtc/RenderSynchronizer.java

diff --git a/sdk/android/api/org/webrtc/RenderSynchronizer.java b/sdk/android/api/org/webrtc/RenderSynchronizer.java
index b1ade84..c89f798 100644
--- a/sdk/android/api/org/webrtc/RenderSynchronizer.java
+++ b/sdk/android/api/org/webrtc/RenderSynchronizer.java
@@ -10,8 +10,11 @@
 
 package org.webrtc;
 
+import android.os.Build.VERSION;
+import android.os.Build.VERSION_CODES;
 import android.os.Handler;
 import android.os.Looper;
+import android.os.Trace;
 import android.view.Choreographer;
 import androidx.annotation.GuardedBy;
 import java.util.List;
@@ -100,8 +103,15 @@
     }
   }
 
+  private void traceRenderWindowChange() {
+    if (VERSION.SDK_INT >= VERSION_CODES.Q) {
+      Trace.setCounter("RenderWindow", renderWindowOpen ? 1 : 0);
+    }
+  }
+
   private void openRenderWindow() {
     renderWindowOpen = true;
+    traceRenderWindowChange();
     for (Listener listener : listeners) {
       listener.onRenderWindowOpen();
     }
@@ -109,6 +119,7 @@
 
   private void closeRenderWindow() {
     renderWindowOpen = false;
+    traceRenderWindowChange();
     for (Listener listener : listeners) {
       listener.onRenderWindowClose();
     }