Bug 144384

Summary: [CSS Box Alignment] Unifying alignment data in a single class
Product: WebKit Reporter: Javier Fernandez <jfernandez>
Component: Layout and RenderingAssignee: Javier Fernandez <jfernandez>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, cdumez, commit-queue, darin, hyatt, jfernandez, rego, svillar
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Description Javier Fernandez 2015-04-29 04:37:44 PDT 
The new CSS Box Alignment specification introduces more complex values and syntax for defining alignment properties. Most of the alignment values were just keyword identifiers, but the new syntax allows different combinations of identifiers to determine the alignment behavior (eg. overflow-alignment keyword). 

This bug describes the need of a refactoring of current alignment logic to define one or more classes to store all the alignment fields. This class will replace the unsigned fields in the StyleRareNonInherited class, hence it will increase it sized considerably. We should take that into account when submitting and reviewing patches.
Comment 1 Javier Fernandez 2015-04-29 04:56:00 PDT 
Created attachment 251946 [details]
Patch
Comment 2 Darin Adler 2015-04-29 08:41:23 PDT 
/Volumes/Data/EWS/WebKit/WebKitBuild/Release/WebCore.framework/PrivateHeaders/StyleRareNonInheritedData.h:37:10: fatal error: 'StyleSelfAlignmentData.h' file not found

I suspect this is adding a new header file to the Xcode project but not marking it “Private”, which is needed if it’s used outside WebCore.
Comment 3 Javier Fernandez 2015-04-29 13:40:09 PDT 
Created attachment 251982 [details]
Patch
Comment 4 Dave Hyatt 2015-04-29 16:24:33 PDT 
Comment on attachment 251982 [details]
Patch

r=me
Comment 5 WebKit Commit Bot 2015-04-29 17:15:00 PDT 
Comment on attachment 251982 [details]
Patch

Clearing flags on attachment: 251982

Committed r183591: <http://trac.webkit.org/changeset/183591>
Comment 6 WebKit Commit Bot 2015-04-29 17:15:07 PDT 
All reviewed patches have been landed.  Closing bug.