[Webkit-unassigned] [Bug 15135] REGRESSION (r19843-r19850): Changing a flexbox's contents makes its container scroll to the top

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jun 15 02:55:38 PDT 2009


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


hamaji at chromium.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #31283|                            |review?
               Flag|                            |




------- Comment #6 from hamaji at chromium.org  2009-06-15 02:55 PDT -------
(From update of attachment 31283)
Thanks for the review!

I understood that we cannot add members into RenderBlock. As the delayed
repaint needs some contexts of layout (repainter, repaintBottom and
repaintTop), I created a hash map which holds the context information and is
used only when a flexbox is layouting its children. This hack looks not so
good, but it seems that this kind of technique is often used for flexible box
support (e.g., gOverrideSizeMap in RenderBox.cpp) and this won't consume extra
RAM for HTMLs which have no flexboxes.

Could you take a look again?

Thanks!


-- 
Configure bugmail: https://bugs.webkit.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