[Webkit-unassigned] [Bug 40366] REGRESSION (r59263): Google Docs Drawing is broken, and probably other SVG-based sites

[bugzilla-daemon at webkit.org]

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





--- Comment #7 from Jonathan Kliegman <kliegs at chromium.org>  2010-06-22 14:02:15 PST ---
I've been trying to get a simple test case for this and haven't been able to simplify it enough.

I have seen what you are showing though from my own testing.  Mainly m_layoutRoot being reset while in the middle of a layout.  

One thing I've found is that adding showNode calls into the rendering process for debugging actually fixes the problem.  It looks like the side effect of it you mention (where it sets the attributes) changes the order enough that the node doesn't get skipped.

I'm wondering if just setting a flag to do a full layout if the DOM is changed during a layout might work here?  Its not the most efficient/elegant solution, but it *should* at least give us correctness in this case.  If the case is rare enough that its not an issue then the performance hit won't be too bad.  If this case is common enough to hurt us I expect we'd have seen more reports.  Although we do need to be careful we don't accidentally loop (not sure if that's possible here).

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