EMBED elements w/o SRC attribute

Koos Vriezen koos.vriezen at xs4all.nl
Sat Jun 5 22:41:30 BST 2004


Hi,

We currently don't support OBJECT/EMBED elm w/o an SCR attribute. However
for EMBED there are cases where a  plugin doesn't have an external URL.
Eg, from an innerHTML output on a page linked from
http://portal.omroep.nl/uitzendinggemist choosing a real player:

   <EMBED id="MediaPlayer" name="MediaPlayer" type="audio/x-pn-realaudio-plugin" width="320" height="240" nojava="true" controls="imagewindow" console="RVOCX" />
 <EMBED type="audio/x-pn-realaudio-plugin" src="http://cgi.omroep.nl/cgi-bin/streams?/tv/nos/journaal/bb.laatste.rm" width="320" height="23" autostart="true" nologo="TRUE" nolabels="true" nojava="true" controls="ControlPanel" console="RVOCX" pluginspage="http://europe.real.com/freeplayer_r1p.html" />
 <EMBED type="audio/x-pn-realaudio-plugin" width="320" height="23" nojava="true" controls="StatusBar" console="RVOCX" />

Here there are three EMBEDs with only one having an URL. Although kmplayer doesn't
support this yet (maybe nspluginviewer with realplay plugin does), I do want
to fix this soon. See patch attached for khtml.

Koos
-------------- next part --------------
Index: rendering/render_frames.cpp
===================================================================
RCS file: /home/kde/kdelibs/khtml/rendering/render_frames.cpp,v
retrieving revision 1.178
diff -u -3 -p -r1.178 render_frames.cpp
--- rendering/render_frames.cpp	23 Mar 2004 00:41:53 -0000	1.178
+++ rendering/render_frames.cpp	5 Jun 2004 21:33:19 -0000
@@ -704,7 +704,7 @@ void RenderPartObject::updateWidget()
               }
           }
       }
-      if (url.isEmpty() || !document()->isURLAllowed(url) || !part->requestObject( this, url, serviceType, params ))
+      if ((url.isEmpty() && element()->id() != ID_EMBED) || !document()->isURLAllowed(url) || !part->requestObject( this, url, serviceType, params ))
           objbase->renderAlternative();
       else
           objbase->setLiveConnect(part->liveConnectExtension(this));


More information about the kfm-devel mailing list