[Webkit-unassigned] [Bug 64974] REGRESSION(87526): ASSERT(!needsLayout()) followed by graphical glitches on google charts (svg loaded in iframe)

[bugzilla-daemon at webkit.org]

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





--- Comment #14 from Nikolas Zimmermann <zimmermann at kde.org>  2011-07-26 00:33:13 PST ---
I agree that there's a bug in the code, that's pretty obvious, on a second sight. It only works for a few tests, because the order of paint/layout calls just matches there (though there's still a race).

I'm going to look into a proper solution.

@James: To summarize again, why I felt angry.
The patch landed over 2 months ago. Now a  bug report has been opened, and a fix is attached, that you gave r+, without giving me the time to comment it. It's not that I have ignored this bug report or anything.

I consider this impolite, and others will agree with me.
Also you should rethink whether it's helpful to state that neither the author nor the reviewer has an understanding of the layout code. After all Rob & me have worked with this codebase since a decade now.
That doesn't mean we're producing bug-free code, nor that my code is always regression-free.

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