[webkit-reviews] review requested: [Bug 23536] Auto-generate HTMLElementFactory : [Attachment 27045] Last bits of autogeneration hackery - Updated with Eric's comment

[bugzilla-daemon at webkit.org]

Julien Chaffraix <jchaffraix at pleyo.com> has asked  for review:
Bug 23536: Auto-generate HTMLElementFactory
https://bugs.webkit.org/show_bug.cgi?id=23536

Attachment 27045: Last bits of autogeneration hackery - Updated with Eric's
comment
https://bugs.webkit.org/attachment.cgi?id=27045&action=review

------- Additional Comments from Julien Chaffraix <jchaffraix at pleyo.com>
> Seems image should just pass another parameter.  MapToTagName="img" or
> something.

I used a parameter switch here (I could not find a better name that
mapToTagName so I used it).

> +    doc->setUsesBeforeAfterRules(true);
> 
> looks no worse than it was before, but is still not as elegant as it could
be.
> You should add a FIXME that HTMLQuoteElements should set that flag on
document
> insertion instead of construction.  I bet we could make a test where you
create
> a Quote in one document, adopt it in another, and then before/after rules are

> not turned on for the document which adopted the quote!
> 

I agree with you: my goal was to get it out of HTMLElementFactory.cpp as we
could miss some quote elements but the logic is wrong. I have added the
requested FIXME.