Comment 10 for bug 1207140

Ok there was a bug in my patch that slipped through.

However there still is a problem with embedding code into the externalmedia block.

If I go to a YouTube video and select the 'Embed' option I get to copy code that looks like this:
<iframe width="420" height="315" src="//www.youtube.com/embed/WM5Gwzk3Vfc" frameborder="0" allowfullscreen></iframe>

Adding that to the externalmedia block gives me no video.

However if I go to a YouTube video and select the 'Embed' option and also tick the 'use old embed code' I get to copy code that looks like this:
<object width="420" height="315"><param name="movie" value="//www.youtube.com/v/WM5Gwzk3Vfc?version=3&amp;hl=en_US"></param><param name="allowFullScreen" value="true"></param><param name="allowscriptaccess" value="always"></param><embed src="//www.youtube.com/v/WM5Gwzk3Vfc?version=3&amp;hl=en_US" type="application/x-shockwave-flash" width="420" height="315" allowscriptaccess="always" allowfullscreen="true"></embed></object>

That code does work for me.

So the question is - is it easier to fix the problem where the iframe code doesn't work or to tell everyone to click the 'use old embed code' option. I think having both work is a better choice.