Commit d0c13733 by olly Committed by tonihei

Don't sample VSYNC when not started

PiperOrigin-RevId: 421855453
parent c4a2579b
...@@ -490,7 +490,6 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer { ...@@ -490,7 +490,6 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer {
releaseCodec(); releaseCodec();
} }
eventDispatcher.enabled(decoderCounters); eventDispatcher.enabled(decoderCounters);
frameReleaseHelper.onEnabled();
mayRenderFirstFrameAfterEnableIfNotStarted = mayRenderStartOfStream; mayRenderFirstFrameAfterEnableIfNotStarted = mayRenderStartOfStream;
renderedFirstFrameAfterEnable = false; renderedFirstFrameAfterEnable = false;
} }
...@@ -558,7 +557,6 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer { ...@@ -558,7 +557,6 @@ public class MediaCodecVideoRenderer extends MediaCodecRenderer {
clearReportedVideoSize(); clearReportedVideoSize();
clearRenderedFirstFrame(); clearRenderedFirstFrame();
haveReportedFirstFrameRenderedForCurrentSurface = false; haveReportedFirstFrameRenderedForCurrentSurface = false;
frameReleaseHelper.onDisabled();
tunnelingOnFrameRenderedListener = null; tunnelingOnFrameRenderedListener = null;
try { try {
super.onDisabled(); super.onDisabled();
......
...@@ -149,18 +149,14 @@ public final class VideoFrameReleaseHelper { ...@@ -149,18 +149,14 @@ public final class VideoFrameReleaseHelper {
updateSurfacePlaybackFrameRate(/* forceUpdate= */ true); 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. */ /** Called when the renderer is started. */
public void onStarted() { public void onStarted() {
started = true; started = true;
resetAdjustment(); resetAdjustment();
if (displayHelper != null) {
checkNotNull(vsyncSampler).addObserver();
displayHelper.register(this::updateDefaultDisplayRefreshRateParams);
}
updateSurfacePlaybackFrameRate(/* forceUpdate= */ false); updateSurfacePlaybackFrameRate(/* forceUpdate= */ false);
} }
...@@ -227,15 +223,11 @@ public final class VideoFrameReleaseHelper { ...@@ -227,15 +223,11 @@ public final class VideoFrameReleaseHelper {
/** Called when the renderer is stopped. */ /** Called when the renderer is stopped. */
public void onStopped() { public void onStopped() {
started = false; started = false;
clearSurfaceFrameRate();
}
/** Called when the renderer is disabled. */
public void onDisabled() {
if (displayHelper != null) { if (displayHelper != null) {
displayHelper.unregister(); displayHelper.unregister();
checkNotNull(vsyncSampler).removeObserver(); checkNotNull(vsyncSampler).removeObserver();
} }
clearSurfaceFrameRate();
} }
// Frame release time adjustment. // Frame release time adjustment.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment