[Webkit-unassigned] [Bug 57052] Infinite Re-layout Loop

[bugzilla-daemon at webkit.org]

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





--- Comment #7 from Nate Chapin <japhet at chromium.org>  2011-04-04 12:49:17 PST ---
(From update of attachment 88102)
View in context: https://bugs.webkit.org/attachment.cgi?id=88102&action=review

I don't know this code well enough to know for sure that this is the right solution, though it looks fine to me.

Just a couple nits :)

> Source/WebCore/svg/graphics/SVGImage.cpp:-77
> -        if (m_image && m_image->imageObserver())
> -            m_image->imageObserver()->changedInRect(m_image, r);

Can we remove this override of invalidateContentsAndWindow()?

Also, I believe this is the only place changedInRect() is called, so we should probably remove that as well.

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