[webkit-dev] attached()

Eric Seidel eric at webkit.org
Thu Sep 9 00:00:14 PDT 2010


I'm about to remove the last non-lazy attaching element.
https://bugs.webkit.org/show_bug.cgi?id=45365


I think we can now kill the whole concept of being "attached".

attached() does not necessarily mean you have a renderer.  It just
means that something called attach() or lazyAttach() on you.


Instead, we could change all elements to mark themselves (and their
parents) as needing style recalc during insertedIntoDocument(),
effectively "attaching/lazyAttaching".

(We could also fix the current lazyAttach() code to check to see if a
parent is already marked with a child needing style resolve, and stop
walking to prevent N^2 behavior.)

I'm not sure if it would make sense to keep the "attached" flag around
to make sure the attach logic is never run twice.

-eric


More information about the webkit-dev mailing list