Review Request: QMediaPlayer using Android's MediaPlayer and MediaMetadataRetriever

[BogDan Vatra]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105051/#review14226
-----------------------------------------------------------



src/android/jar/src/org/kde/necessitas/mobile/QtMediaPlayer.java
<http://git.reviewboard.kde.org/r/105051/#comment11237>

    Please add an email address where me and other people can contact you.



src/android/jar/src/org/kde/necessitas/mobile/QtMediaPlayerController.java
<http://git.reviewboard.kde.org/r/105051/#comment11238>

    Please add an email address where me and other people can contact you.



src/android/jar_14/src/org/kde/necessitas/mobile/QtMediaMetadataRetriever.java
<http://git.reviewboard.kde.org/r/105051/#comment11239>

    Please add an email address where me and other people can contact you.



src/android/jar_14/src/org/kde/necessitas/mobile/QtMediaMetadataRetrieverController.java
<http://git.reviewboard.kde.org/r/105051/#comment11240>

    Please add an email address where me and other people can contact you.


- BogDan Vatra


On May 25, 2012, 4:59 p.m., Gabi Julien wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105051/
> -----------------------------------------------------------
> 
> (Updated May 25, 2012, 4:59 p.m.)
> 
> 
> Review request for Necessitas.
> 
> 
> Description
> -------
> 
> As discussed on the mailing list[1], this is an implementation of QMediaPlayer using Android's MediaPlayer and MediaMetadataRetriever classes. This patch is only the android-lighthouse part. Another patch on qt-mobility is needed.
> 
> [1] http://mail.kde.org/pipermail/necessitas-devel/2012-May/000916.html
> 
> 
> Diffs
> -----
> 
>   src/android/jar/src/org/kde/necessitas/mobile/QtMediaPlayer.java 292e8cc 
>   src/android/jar/src/org/kde/necessitas/mobile/QtMediaPlayerController.java PRE-CREATION 
>   src/android/jar_14/src/org/kde/necessitas/mobile/QtMediaMetadataRetriever.java PRE-CREATION 
>   src/android/jar_14/src/org/kde/necessitas/mobile/QtMediaMetadataRetrieverController.java PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/105051/diff/
> 
> 
> Testing
> -------
> 
> Using Asus's Transformer Prime TF 201
> 1. 1080p h264 video on Android 4.0.3.
> 2. mp3 music without surfaces.
> 3. Various tests involving volume, seeking, duration,... using the QMediaPlayer API.
> 
> My tests on Android 2.3.1 failed so far because a second hardware accelerated surface is needed. Also, the MediaMetadataRetriever is not available before API 10. The good news is that music playback should still work.
> 
> 
> Thanks,
> 
> Gabi Julien
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/necessitas-devel/attachments/20120528/0459ed9f/attachment.html>