Review Request 106671: kjs/khtml: Reuse existing html5 Audio Element instead of duplicating it

Commit Hook null at kde.org
Sat Mar 30 15:14:43 GMT 2013


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

(Updated March 30, 2013, 3:14 p.m.)


Status
------

This change has been marked as submitted.


Review request for kdelibs.


Description
-------

kjs/khtml: Reuse existing html5 Audio Element instead of duplicating it

that adds all html functionality to the javascript Audio.
Previous one just had pause/play/stop/omload/onerror and
NOTE "stop" is not even part of the HTMLAudioElement/HTMLMediaElement(.idl), so its removed now.


Diffs
-----

  khtml/CMakeLists.txt 99034cc 
  khtml/ecma/kjs_audio.h 956d72a 
  khtml/ecma/kjs_audio.cpp f67cdc9 

Diff: http://git.reviewboard.kde.org/r/106671/diff/


Testing
-------

Makes websites like
http://release.blackmesasource.com/?sm2-preferflash=false
and
http://9elements.com/html5demos/audio/
from the javascript point of view

not 100% as the html element is still missing some implementations
e.g. scheduleLoad and canplay event


Thanks,

Bernd Buschinski

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130330/5a1ab562/attachment.htm>


More information about the kde-core-devel mailing list