Please review: parsing HTML 5 media elements

Hi!

Germain had added support for the <audioand <videoelements to the KDE
4.1. feature plan. Not sure whether he already did some work in his git
repo. As the freeze is coming near I went ahead and wanted to lay the
groundwork for them. Details can be implemented later.

I'd like to ask either Allen or Germain to take a look at the necessary
parser extension. For better readability he attached patch does not
contain the generated dtd.h and dtd.c code. The element creation code is
just a dummy printing out a message.

Harri.

P.S. Note to bystanders: we already have audio support in KHTML for some
time now. But that was based on the "Audio" JS element. Equally
non-standard as <audioso far

Le vendredi 16 mai 2008, Harri Porten a ©crit :
Hi!
>
Germain had added support for the <audioand <videoelements to the KDE
4.1. feature plan. Not sure whether he already did some work in his git
repo.

No, I've been too busy with designMode to have a go at those.

As the freeze is coming near I went ahead and wanted to lay the
groundwork for them. Details can be implemented later.
>
I'd like to ask either Allen or Germain to take a look at the necessary
parser extension. For better readability he attached patch does not
contain the generated dtd.h and dtd.c code. The element creation code is
just a dummy printing out a message.

I don't see anything wrong with the patch

Greetings,
Germain