Unlike playIndex, readMedia adds the media to the VLC playlist (and thus
to the history) which is wrong in this case.

Thanks to the last minute fixes ;)
This reverts commit 38de25d3.

onResume is enough.
also no need to use special code when resuming from pause, load() do all the necessary checks.

There is a regression in gcc 4.6, DECL_USER_ALIGN is not handled properly,
 variables are not aligned, causing a crash in avcodec when decoding aac sbr with neon.

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52181

Alexandre Perraud authored 12 years ago and Jean-Baptiste Kempf committed 12 years ago

Signed-off-by: Jean-Baptiste Kempf <jb@videolan.org>

Instead of pausing the playback in onPause we stop it completely to
prevent the output thread from running while the application is in the
background. The playback is then restarted at the same position in the
onResume method.

The allocated memory used to store the thumbnail was insufficient and
caused a memory corruption on the Nexus7 tablet.

Tested on froyo (2.2)

Android apparently does an "OR" match, but if there is no mimeType it doesn't match, so we need to have the parallel structure.

I tested this on 4.1 with a wide variety of protocols, with and without types and they all work.

This allows us to show controls for the audio player on the lock screen, a popularly demanded feature.

Close #6993