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

[bugzilla-daemon at webkit.org]

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


hyatt at apple.com changed:

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




------- Comment #10 from hyatt at apple.com  2009-06-15 23:16 PDT -------
(From update of attachment 31284)
You've picked a very challenging bug to work on here.

I'm not sure I'd worry so much about repaint being wrong.  If the flexing block
has kids that move during the first layout, your patch isn't going to help that
case.  They are still going to repaint too much.  I think you should just
concentrate on the scrolling issue and ignore repaint for now.

As for scrolling, trying to stop just the immediate child block from updating
its scroll info during the first layout doesn't really generally solve the
problem.  What about a scrollable object inside a flexing block that
incorrectly adjusts its scroll position in response to the first layout?

I think a simpler fix for this bug is to just make the scroll position updating
code immune to flex box layout thrashing.


-- 
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.