Bug 144384 - [CSS Box Alignment] Unifying alignment data in a single class
Summary: [CSS Box Alignment] Unifying alignment data in a single class
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Javier Fernandez
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-29 04:37 PDT by Javier Fernandez
Modified: 2015-04-29 17:15 PDT (History)
8 users (show)

See Also:


Attachments
Patch (46.28 KB, patch)
2015-04-29 04:56 PDT, Javier Fernandez
no flags Details | Formatted Diff | Diff
Patch (50.05 KB, patch)
2015-04-29 13:40 PDT, Javier Fernandez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.