Create media session UI and fix player notification #8678

The player is now passed directly, it made no sense to wrap around it in a callback that was not really a callback but rather, actually, a wrapper.

The thumbnail was not being updated in the media session metadata after it was loaded, since there was no metadata update in that case, only a notification update.

Before the thumbnail finishes loading for the new video the player is now playing, the old thumbnail was being used, leading to wrong thumbnails set in the media session and the notification.

This makes sure the thumbnail is removed from the notification if the user disables thumbnails

A while ago NewPipe called the metadata update function very often, so checks were needed to ensure not wasting time updating metadata if it were already up to date. Now, instead, the metadata update function is called exactly when needed, i.e. when metadata changes, so such checks are not needed anymore (and were probably also a little resource-heavy).

Though still make sure metadata is updated after the thumbnail is loaded. This fixes the wrong seekbar properties (duration and current position) being shown in the notification sometimes.

Before the Target would sometimes be garbage collected before being called with the loaded thumbnail, since Picasso holds weak references to targets

+ Automatically fixed code style (imports)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Create media session UI and fix player notification #8678

Create media session UI and fix player notification #8678

Commits on Aug 25, 2022