Fixes #2398

So it's now completely standalone

Steve Lhomme authored 4 months ago and Nicolas Pomepuy committed 3 months ago

With the same SHA512SUM check.

Duncan McNamara authored 3 months ago and Nicolas Pomepuy committed 3 months ago

This is to use getMedia and get a MediaWrapper with meta data from the
medialibrary instead of a bland/empty MediaWrapper.

When opening a media from an other app, if the media wasn't previously
indexed, getMedia will return null, hence the additional use of
getAbstractMediaWrapper(uri). If not the player would just stop without
any warning.

Duncan McNamara authored 3 months ago and Nicolas Pomepuy committed 3 months ago

Some browsers won't prefix intent types with "video", if a video is
already playing in background, it will play the new video in background.
Because of this, it is safer to stop the player, causing 'isPlaying' to
go false in playIndex(...) and not keeping the previous state.

Fixes #3114

On 757baf34 I forgot two instances.

Nicolas Pomepuy authored 3 months ago and Duncan McNamara committed 3 months ago

Fixes #3072