-
Notifications
You must be signed in to change notification settings - Fork 2.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(android, exoplayer): ensure playback progress is reported #2664
fix(android, exoplayer): ensure playback progress is reported #2664
Conversation
Thanks! As soon as we get it reviewed it will be merged! |
private final Handler progressHandler = new Handler() { | ||
@Override | ||
public void handleMessage(Message msg) { | ||
switch (msg.what) { | ||
case SHOW_PROGRESS: | ||
if (player != null |
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.
Why did you remove this null check? Wouldn't it be safer to leave it there? (I'm OK with removing the state checks, but null check might be beneficial here).
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 fully agree ! change done
…native-video into fix/report_time_position_when_updated # Conflicts: # CHANGELOG.md # android-exoplayer/src/main/java/com/brentvatne/exoplayer/ReactExoplayerView.java
# Conflicts: # CHANGELOG.md
…native-video into fix/report_time_position_when_updated # Conflicts: # CHANGELOG.md
…eelSmart/react-native-video into fix/report_time_position_when_updated
Just fix conflict, ready to merge |
fix(android, exoplayer): ensure playback progress is reported even if player is paused
Adresses #2141 on Android.
and shall fix: #2372
Provide an example of how to test the change
A live channel sample is available in basic sample.
On live stream, position indicator keeps updating after playback is paused.
Focus area
You can ensure that position is not reported too often
Describe the changes