[vlc-devel] [PATCH] - Updated the browser plugins to use the Media List Player for playlist traversal

Niles Bindel zaggal69 at gmail.com
Fri Sep 4 22:49:52 CEST 2009


I've separated the patch into two stages...the first being the conversion to
Media List Player and the second a cosmetic patch to be applied after the
first patch.  I also found that my first patch was missing the Mozilla
plugin changes I made so I've added those changes to these patches as well.

I also tried to unify more of the functionality between the plugins so that
functions available in Mozilla are now available in ActiveX, such as
"player_has_vout" although I did not go into the control classes to have
them route through the plugin class like the Mozilla plugin does.

> I haven't yet looked at the benefits of that, but I'd like you to split

Benefits:
1.  Corrects the previously mentioned bug where the next playlist items are
not played upon reaching the end of currently playing file.
2.  Removes unnecessary duplication of basic playlist functionality from
plugins and centralizes it in the Media List Player.
3.  Will allow for easy integration of new and future features added to the
Media List Player such as playback modes (repeat, loop, etc).
4.  Proper media list traversal when media sub-items exist.

> And if you feel like it you could also (in yet anoter separate patch)
> use the exception-free error reporting API.

I have no idea what you are referring to.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20090904/92e1560d/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-This-patch-updates-the-browser-plugins-to-use-the-Me.patch
Type: application/octet-stream
Size: 26499 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20090904/92e1560d/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Cosmetic-changes-to-browser-plugin-files.patch
Type: application/octet-stream
Size: 30049 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20090904/92e1560d/attachment-0001.obj>


More information about the vlc-devel mailing list