GNOME Bugzilla – Bug 345554
Support video/divx embeds
Last modified: 2006-09-07 22:11:49 UTC
+++ This bug was initially created as a clone of Bug #333074 +++ File doesn't play in Mozilla: http://community.divx.com/movies/details.php?id=228 because the server-side detection code is slightly broken. Mailed webmaster@divx.com about it.
The detection has been "fixed". Now it only relies on the plugin name[1]. To implement the Javascript interface, this SDK should contain the docs: http://download.divx.com/player/DivXWebPlayer_WebmasterSDK.zip And another example video is at: http://stage6.divx.com/members/66592/videos/1006708 [1]: http://stage6.divx.com/javascripts/content.js?1157157616
Created attachment 72252 [details] [review] totem-mully-plugin.patch Only things to fix are the UUID of the Javascript component, and the Javascript interface itself.
+[scriptable, uuid(aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa)] Use 'uuidgen' here (apt-get install libuuid1) +/* 67DABFBF-D0AB-41fa-9C46-CC0F21721616 */ +static const nsCID kClassID = + { 0x67DABFBF, 0xD0AB, 0x41fa, + { 0x9C, 0x46, 0xCC, 0x0F, 0x21, 0x72, 0x16, 0x16 } }; The others use lowercase hex here... btw you can generate this with the script at http://www.gnome.org/~chpe/programs/uuidgen-c++ (found somewhere on a blog from planet.mozilla.org, don't remember which one)
(In reply to comment #3) > +[scriptable, uuid(aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa)] > > Use 'uuidgen' here (apt-get install libuuid1) I wanted to use the same as the one provided by the DivX plugin, but I didn't know whether it would make any difference. > +/* 67DABFBF-D0AB-41fa-9C46-CC0F21721616 */ > +static const nsCID kClassID = > + { 0x67DABFBF, 0xD0AB, 0x41fa, > + { 0x9C, 0x46, 0xCC, 0x0F, 0x21, 0x72, 0x16, 0x16 } }; > > The others use lowercase hex here... btw you can generate this with the script > at > http://www.gnome.org/~chpe/programs/uuidgen-c++ (found somewhere on a blog from > planet.mozilla.org, don't remember which one) OK, will change that.
(In reply to comment #4) > (In reply to comment #3) > > +[scriptable, uuid(aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa)] > > > > Use 'uuidgen' here (apt-get install libuuid1) > > I wanted to use the same as the one provided by the DivX plugin, but I didn't > know whether it would make any difference. I've no idea if the IID of an object is even accessible to content JS... I think it's fine to use a different IID until proven otherwise :)
2006-09-07 Bastien Nocera <hadess@hadess.net> * browser-plugin/Makefile.am: * browser-plugin/idl/Makefile.am: * browser-plugin/idl/totemIMullYPlayer.idl: * browser-plugin/totemGMPPlugin.cpp: * browser-plugin/totemMullYPlugin.cpp: * browser-plugin/totemMullYPlugin.h: * browser-plugin/totemPlugin.cpp: * configure.in: Add a new "MullY" plugin, to handle videos on the DivX website, masquerading as the "DivX Web Player" (Closes: #345554)