From ff7acb19a16d9604cf3e54f2cd4ebfa6cb5e8fc6 Mon Sep 17 00:00:00 2001 From: tserng Date: Fri, 14 Jul 2017 02:35:53 -0700 Subject: [PATCH] Reset isFirstFrameRendered on init of SurfaceViewRenderer If a SurfaceViewRenderer is reinitialized, the onFirstFrameRendered callback is not fired. Ensure that we reset the flag when the SurfaceViewRenderer is initialized. BUG=webrtc:7985 Review-Url: https://codereview.webrtc.org/2981793002 Cr-Commit-Position: refs/heads/master@{#19016} --- AUTHORS | 1 + webrtc/sdk/android/api/org/webrtc/SurfaceViewRenderer.java | 1 + 2 files changed, 2 insertions(+) diff --git a/AUTHORS b/AUTHORS index 3a1735167f..8c7e4d7be1 100644 --- a/AUTHORS +++ b/AUTHORS @@ -10,6 +10,7 @@ Anil Kumar Ben Strong Bob Withers Bridger Maxwell +Chris Tserng Christophe Dumez Cody Barnes Colin Plumb diff --git a/webrtc/sdk/android/api/org/webrtc/SurfaceViewRenderer.java b/webrtc/sdk/android/api/org/webrtc/SurfaceViewRenderer.java index ccfb9d0641..67b41fa9f9 100644 --- a/webrtc/sdk/android/api/org/webrtc/SurfaceViewRenderer.java +++ b/webrtc/sdk/android/api/org/webrtc/SurfaceViewRenderer.java @@ -92,6 +92,7 @@ public class SurfaceViewRenderer ThreadUtils.checkIsOnMainThread(); this.rendererEvents = rendererEvents; synchronized (layoutLock) { + isFirstFrameRendered = false; rotatedFrameWidth = 0; rotatedFrameHeight = 0; frameRotation = 0;