[Webkit-unassigned] [Bug 69605] CSS Counters have wrong values

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Oct 7 17:21:53 PDT 2011


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





--- Comment #5 from Darin Adler <darin at apple.com>  2011-10-07 17:21:53 PST ---
(From update of attachment 110234)
View in context: https://bugs.webkit.org/attachment.cgi?id=110234&action=review

> Source/WebCore/rendering/RenderObject.cpp:452
> +RenderObject* RenderObject::rendererOfAfterPseudoElement()
> +{
> +    RenderObject* lastContinuation = this;
> +    while (RenderObject* continuation = lastContinuation->virtualContinuation())
> +        lastContinuation = continuation;
> +    return lastContinuation->afterPseudoElementRenderer();
> +}

This code is unnecessarily inefficient. Only the first call needs to be virtualContinuation. Once we have called it once we can call the non-virtual continuation function.

Does this really need to be a member function? Maybe we should just use a helper function in RenderCounter.cpp and not touch the RenderObject.h/cpp source file at all.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list