[Webkit-unassigned] [Bug 23940] Use Document::createElement(const QualifiedName&, bool) when creating a known element inside WebCore

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Feb 18 01:50:24 PST 2009


https://bugs.webkit.org/show_bug.cgi?id=23940





------- Comment #5 from jchaffraix at webkit.org  2009-02-18 01:50 PDT -------
> > the only problematic
> > change I see is in DragController - the other changes are (X)HTML-specific.
> 
> I think DragController is fine, too -the results was casted to
> HTMLAnchorElement* anyway. I wonder if the cast can be avoided altogether by
> creating the element more directly.

Right, I have missed the cast. I will make this change too as a short-term
enhancement but I feel more and more that such manual creation are bad and will
backfire on us someday. We should have the Elements' factories exports a way to
create a "default" element using the static names.

> Let's change it back then - from my understanding of the code,
> "createdByParser" is tightly coupled to the parser behavior, which is unlikely
> to be followed here.

Ok.


-- 
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