Don't sample VSYNC when not started

PiperOrigin-RevId: 421855453
This commit is contained in:
olly 2022-01-14 18:10:39 +00:00 committed by Ian Baker
parent c6e5ace310
commit 9e4d68ecb6
2 changed files with 5 additions and 15 deletions

View file

@ -493,7 +493,6 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer {
releaseCodec();
}
eventDispatcher.enabled(decoderCounters);
frameReleaseHelper.onEnabled();
mayRenderFirstFrameAfterEnableIfNotStarted = mayRenderStartOfStream;
renderedFirstFrameAfterEnable = false;
}
@ -561,7 +560,6 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer {
clearReportedVideoSize();
clearRenderedFirstFrame();
haveReportedFirstFrameRenderedForCurrentSurface = false;
frameReleaseHelper.onDisabled();
tunnelingOnFrameRenderedListener = null;
try {
super.onDisabled();

View file

@ -151,18 +151,14 @@ public final class VideoFrameReleaseHelper {
updateSurfacePlaybackFrameRate(/* forceUpdate= */ true);
}
/** Called when the renderer is enabled. */
public void onEnabled() {
if (displayHelper != null) {
checkNotNull(vsyncSampler).addObserver();
displayHelper.register(this::updateDefaultDisplayRefreshRateParams);
}
}
/** Called when the renderer is started. */
public void onStarted() {
started = true;
resetAdjustment();
if (displayHelper != null) {
checkNotNull(vsyncSampler).addObserver();
displayHelper.register(this::updateDefaultDisplayRefreshRateParams);
}
updateSurfacePlaybackFrameRate(/* forceUpdate= */ false);
}
@ -229,15 +225,11 @@ public final class VideoFrameReleaseHelper {
/** Called when the renderer is stopped. */
public void onStopped() {
started = false;
clearSurfaceFrameRate();
}
/** Called when the renderer is disabled. */
public void onDisabled() {
if (displayHelper != null) {
displayHelper.unregister();
checkNotNull(vsyncSampler).removeObserver();
}
clearSurfaceFrameRate();
}
// Frame release time adjustment.