[Webkit-unassigned] [Bug 8739] Crash in RenderTableSection::paint due to manipulating DOM on resize

bugzilla-daemon at opendarwin.org bugzilla-daemon at opendarwin.org
Sat May 13 03:25:32 PDT 2006


------- Comment #10 from opendarwin.org at mitzpettel.com  2006-05-13 03:25 PDT -------
(In reply to comment #9)
> With your patch, though, we do one layout, resize, and then do another layout. 
> Couldn't we do the resize check at the top of the function, and then just let
> the remaining code do the single layout?

So we talked about this on IRC. It's true that this fix add a layout, but only
in the resize-handler-that-manupulates-the-DOM case.

If you want to do just one layout, you need to go deep into
Frame::forceLayoutWithPageWidthRange and separate it into a function that sets
the width and marks as needing layout (which you will call into before sending
the resize event) and a function that forces layout (possibly twice for
printing?) (which you will call into after sending the resize event). I'm not
sure it's worth it.

Configure bugmail: http://bugzilla.opendarwin.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

More information about the webkit-unassigned mailing list