-
Notifications
You must be signed in to change notification settings - Fork 28
Conversation
@Override | ||
public void onDroppedFrames(int count, long elapsedMs) { | ||
for (VideoRendererEventListener listener : listeners) { | ||
listener.onDroppedFrames(count, elapsedMs); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we do not have this callback anymore?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No we don't. The only place it is exposed by exoplayer
is on the VideoRendererEventListener
, which can't be attached to exoplayer directly any more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Mecharyry do we still have this info in the analytics listener
? or now we cannot know if we have dropped frames at all?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah yes, onDroppedVideoFrames
on exoplayer's AnalyticsListener is the one we want.
} | ||
|
||
@Override | ||
public void onVideoDecoderInitialized(String decoderName, long initializedTimestampMs, long initializationDurationMs) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we do not have this callback anymore? and others...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are exposed through the analytics listener
now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm happy assuming that all instances of the VideoRendererListener
have been removed 👍
Problem
The method on
exoplayer
to attach aVideoRendererEventListener
has been deprecated. That means we weren't attaching ourVideoSizeChangedListener
properly.Solution
Change the forwarder to implement exoplayer's new
VideoListener
and attach it in the facade.Test(s) added
I'm not sure there's any worthwhile tests to add here. What we really need is the capability to say, when we've created a player, and that player sends an
onVideoSizeChanged
event, our event listeners all hear it. However, it's not possible to write a test like that right now. It would also be quite an undertaking as we'd need a fake exoplayer which is kept up to date with the version we're targeting.Paired with
No one