-
Notifications
You must be signed in to change notification settings - Fork 4.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
[expo-av] Fix exception on Android when loading invalid Video source #9596
Conversation
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.
Almost there but looks good to me
} | ||
|
||
mPlayerData.setStatusUpdateListener(mStatusUpdateListener); | ||
|
||
if (mMediaController == null) { | ||
mMediaController = new MediaController(VideoView.this.getContext()); | ||
} |
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.
Run linter to parry this, update changeLog and this should be good to go
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.
Thanks for taking a look! Updated the changelog 👍
Run linter to parry this
Please elaborate
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.
LGTM 👍
…xpo#9596) * [expo-av] Fix exception on Android when loading invalid Video source * [expo-av] Update changelog
Why
Fix invalid state when loading a source fails. This leads to an exception when loading a new source.
Fixes #9185
How
When an error occurred, the MediaController was cleaned up. Upon loading a new source after that, an exception occurred because the
mMediaController
variable was null.Test Plan
Steps to reproduce: