[webkit-reviews] review requested: [Bug 64974] REGRESSION(87526): ASSERT(!needsLayout()) followed by graphical glitches on google charts (svg loaded in iframe) : [Attachment 103049] Patch v7

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Aug 5 01:16:36 PDT 2011


Nikolas Zimmermann <zimmermann at kde.org> has asked  for review:
Bug 64974: REGRESSION(87526): ASSERT(!needsLayout()) followed by graphical
glitches on google charts (svg loaded in iframe)
https://bugs.webkit.org/show_bug.cgi?id=64974

Attachment 103049: Patch v7
https://bugs.webkit.org/attachment.cgi?id=103049&action=review

------- Additional Comments from Nikolas Zimmermann <zimmermann at kde.org>
Fix build, as discussed on IRC. De-inline embeddedContentBox() again, it's used
from other translation units as well. I specifically want to avoid moving
embeddedContentBox() inline into the header to avoid having to pull in various
includes in FrameView.h.


More information about the webkit-reviews mailing list