loliball/media
1
0
Fork 0
mirror of https://github.com/androidx/media.git synced 2025-04-30 06:46:50 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Remove unused parameter from VideoFrameRenderControl.FrameRenderer

Browse Source 
PiperOrigin-RevId: 689281689
This commit is contained in:
kimvde 2024-10-24 01:24:50 -07:00 committed by Copybara-Service
parent b5615d5e91
commit d25a423888
3 changed files with 4 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/PlaybackVideoGraphWrapper.java
View File

@ -926,10 +926,7 @@ public final class PlaybackVideoGraphWrapper implements VideoSinkProvider, Video
@Override
public void renderFrame(
long renderTimeNs,
long bufferPresentationTimeUs,
long streamOffsetUs,
boolean isFirstFrame) {
long renderTimeNs, long bufferPresentationTimeUs, boolean isFirstFrame) {
if (isFirstFrame && currentSurfaceAndSize != null) {
for (PlaybackVideoGraphWrapper.Listener listener : listeners) {
listener.onFirstFrameRendered(PlaybackVideoGraphWrapper.this);

9
libraries/exoplayer/src/main/java/androidx/media3/exoplayer/video/VideoFrameRenderControl.java
View File

@ -49,11 +49,9 @@ import androidx.media3.exoplayer.ExoPlaybackException;
* rendered immediately.
* @param presentationTimeUs The frame's presentation time, in microseconds, which was announced
* with {@link VideoFrameRenderControl#onOutputFrameAvailableForRendering(long)}.
* @param streamOffsetUs The stream offset, in microseconds, that is associated with this frame.
* @param isFirstFrame Whether this is the first frame of the stream.
*/
void renderFrame(
long renderTimeNs, long presentationTimeUs, long streamOffsetUs, boolean isFirstFrame);
void renderFrame(long renderTimeNs, long presentationTimeUs, boolean isFirstFrame);
/**
* Called to drop the {@linkplain
@ -218,10 +216,7 @@ import androidx.media3.exoplayer.ExoPlaybackException;
? VideoFrameProcessor.RENDER_OUTPUT_FRAME_IMMEDIATELY
: videoFrameReleaseInfo.getReleaseTimeNs();
frameRenderer.renderFrame(
renderTimeNs,
presentationTimeUs,
outputStreamOffsetUs,
videoFrameReleaseControl.onFrameReleasedIsFirstFrame());
renderTimeNs, presentationTimeUs, videoFrameReleaseControl.onFrameReleasedIsFirstFrame());
}
private boolean maybeUpdateOutputStreamOffset(long presentationTimeUs) {

10
libraries/exoplayer/src/test/java/androidx/media3/exoplayer/video/VideoFrameRenderControlTest.java
View File

@ -60,7 +60,6 @@ public class VideoFrameRenderControlTest {
.renderFrame(
/* renderTimeNs= */ anyLong(),
/* presentationTimeUs= */ eq(0L),
/* streamOffsetUs= */ eq(0L),
/* isFirstFrame= */ eq(true));
}
@ -92,7 +91,6 @@ public class VideoFrameRenderControlTest {
.renderFrame(
/* renderTimeNs= */ anyLong(),
/* presentationTimeUs= */ eq(0L),
/* streamOffsetUs= */ eq(0L),
/* isFirstFrame= */ eq(true));
inOrder.verifyNoMoreInteractions();
@ -106,7 +104,6 @@ public class VideoFrameRenderControlTest {
.renderFrame(
/* renderTimeNs= */ anyLong(),
/* presentationTimeUs= */ eq(10_000L),
/* streamOffsetUs= */ eq(0L),
/* isFirstFrame= */ eq(false));
inOrder.verifyNoMoreInteractions();
}
@ -135,7 +132,7 @@ public class VideoFrameRenderControlTest {
.verify(frameRenderer)
.onVideoSizeChanged(new VideoSize(/* width= */ VIDEO_WIDTH, /* height= */ VIDEO_HEIGHT));
// First frame has the first stream offset.
inOrder.verify(frameRenderer).renderFrame(anyLong(), eq(0L), eq(10_000L), eq(true));
inOrder.verify(frameRenderer).renderFrame(anyLong(), eq(0L), eq(true));
inOrder.verifyNoMoreInteractions();
// 10 milliseconds pass
@ -151,7 +148,6 @@ public class VideoFrameRenderControlTest {
.renderFrame(
/* renderTimeNs= */ anyLong(),
/* presentationTimeUs= */ eq(10_000L),
/* streamOffsetUs= */ eq(20_000L),
/* isFirstFrame= */ eq(true));
inOrder.verifyNoMoreInteractions();
}
@ -189,7 +185,6 @@ public class VideoFrameRenderControlTest {
.renderFrame(
/* renderTimeNs= */ anyLong(),
/* presentationTimeUs= */ eq(0L),
/* streamOffsetUs= */ eq(0L),
/* isFirstFrame= */ eq(true));
inOrder.verifyNoMoreInteractions();
@ -233,7 +228,6 @@ public class VideoFrameRenderControlTest {
.renderFrame(
/* renderTimeNs= */ anyLong(),
/* presentationTimeUs= */ eq(10_000L),
/* streamOffsetUs= */ eq(0L),
/* isFirstFrame= */ eq(true));
inOrder.verifyNoMoreInteractions();
}
@ -271,7 +265,6 @@ public class VideoFrameRenderControlTest {
.renderFrame(
/* renderTimeNs= */ anyLong(),
/* presentationTimeUs= */ eq(0L),
/* streamOffsetUs= */ eq(0L),
/* isFirstFrame= */ eq(true));
assertThat(videoFrameRenderControl.hasReleasedFrame(/* presentationTimeUs= */ 0)).isTrue();
}
@ -299,7 +292,6 @@ public class VideoFrameRenderControlTest {
.renderFrame(
/* renderTimeNs= */ anyLong(),
/* presentationTimeUs= */ eq(0L),
/* streamOffsetUs= */ eq(0L),
/* isFirstFrame= */ eq(true));
videoFrameRenderControl.flush();