Bug 88366

Summary: Changing flexbox justify/alignment doesn't update their positioning
Product: WebKit Reporter: Eric <ericbidelman>
Component: Layout and RenderingAssignee: Tony Chang <tony>
Severity: Normal CC: eric, ojan, tony, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 62048    
Description Flags
Test case
Patch none

Description Eric 2012-06-05 14:57:15 PDT
The attached test case demonstrates this. The buttons update the justify/alignment properties
of the flex container, but the items are not repositioned correctly. Toggling off the original
style set on the flex contain (via the inspector) works. It appears a repaint is happening.

I'm on 536.5 (@118056)
Comment 1 Eric 2012-06-05 14:59:46 PDT
Created attachment 145875 [details]
Test case
Comment 2 Tony Chang 2012-06-05 15:10:28 PDT
I don't have time to investigate this right now, but I suspect that the bug is that in RenderStyle::diff, we don't compare the new values on rareNonInheritedData.

Previously, we would compare rareNonInheritedData->m_flexibleBox against each other, but moving the values to rareNonInheritedData doesn't automatically do this.  Once we realize that the style has changed, we'll probably re-layout properly.

The fix is probably around line RenderStyle.cpp:376.
Comment 3 Tony Chang 2012-06-05 17:35:03 PDT
Created attachment 145903 [details]
Comment 4 Tony Chang 2012-06-05 17:35:27 PDT
I wasn't making progress on my other bug, so I decided to fix this :)
Comment 5 WebKit Review Bot 2012-06-05 23:45:40 PDT
Comment on attachment 145903 [details]

Clearing flags on attachment: 145903

Committed r119564: <http://trac.webkit.org/changeset/119564>
Comment 6 WebKit Review Bot 2012-06-05 23:45:50 PDT
All reviewed patches have been landed.  Closing bug.