[Webkit-unassigned] [Bug 22441] Bridge the gap between the generated ElementFactory and HTMLElementFactory
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Nov 24 15:34:02 PST 2008
https://bugs.webkit.org/show_bug.cgi?id=22441
------- Comment #11 from jchaffraix at pleyo.com 2008-11-24 15:34 PDT -------
(In reply to comment #10)
> (From update of attachment 25453 [review])
> > - WebCore::Element* createElement(const WebCore::QualifiedName& qName, WebCore::Document* doc, bool createdByParser = true);
> > - static $parameters{'namespace'}Element* create$parameters{'namespace'}Element(const WebCore::QualifiedName& qName, WebCore::Document* doc, bool createdByParser = true);
> > + PassRefPtr<Element> createElement(const WebCore::QualifiedName&, WebCore::Document*, bool createdByParser = true);
> > + static PassRefPtr<$parameters{'namespace'}Element> create$parameters{'namespace'}Element(const WebCore::QualifiedName&, WebCore::Document*, bool createdByParser = true);
>
> I don't understand how you can change the declaration of the createElement
> function here without also changing the return type in the definition of
> createElement. I guess that's just a placeholder for the future.
FYI, createElement is never implemented and is indeed a placeholder related to
a comment in make_names.pl
(http://trac.webkit.org/browser/trunk/WebCore/dom/make_names.pl#L633). I have
weighted about removing it from both make_names.pl and HTMLElementFactory but I
liked the idea of leaving this FIXME.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
More information about the webkit-unassigned
mailing list