Set the title for external video players

[epitron]

This makes the external player experience nicer.

I had to set the title twice on the intent, since "Intent.EXTRA_TITLE" didn't work on MX Player. (MX Player's supported intent data is here: https://sites.google.com/site/mxvpen/api).

I think I did that correctly. Anyhow, it works. :)

[theScrabi]

Ok I see if i can also apply that to audio streaming. :) Thanks for the help.

[epitron]

Np!

[theScrabi]

Well i could set the title for music streams like you set it for video streams. But all it does is showing the title in the video player activity of VLC. In the audio only activity it displays the url again.

[epitron]

I guess each player has specific quirks.

This looks like the point where VLC is getting the title from the intent: https://github.com/mstorsjo/vlc-android/blob/master/vlc-android/src/org/videolan/vlc/gui/video/VideoPlayerActivity.java#L1898

It seems to only check for the intent extras when getAction().equals(PLAY_FROM_VIDEOGRID). I'm not sure what most of this function is doing, but maybe audio is a different action?

[theScrabi]

That might be it.
Well I don't like VLC for streaming anyway. It has bad audio quality. So I somehow one day want to make a player for NP itself.

[sonarqubecloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
10 Code Smells

0.0% Coverage
0.0% Duplication

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[sonarqubecloud]

Quality Gate failed

Failed conditions
6.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud