Bug 142329

Summary: [CSS Grid Layout] Using automatic (instead of min-content) minimums for 'auto' tracks
Product: WebKit Reporter: Sergio Villar Senin <svillar>
Component: CSSAssignee: Sergio Villar Senin <svillar>
Status: RESOLVED FIXED    
Severity: Normal CC: benjamin, darin, hyatt, kling, koivisto, rego
Priority: P2 Keywords: BlinkMergeCandidate
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 146018    
Bug Blocks: 60731    
Attachments:
Description Flags
Patch darin: review+

Description Sergio Villar Senin 2015-03-05 01:40:58 PST 
This has recently changed in the specs. See https://lists.w3.org/Archives/Public/www-style/2015Jan/0166.html for more details.

The most important part is the following:

    More precisely (syntax-wise), this would allow 'auto' to be used within the
    minmax() function (it's currently forbidden) and have the 'auto' keyword map
    to minmax(auto, auto) instead of minmax(min-content, max-content).
        * As a minimum, 'auto' would mean "use the specified minimum size, or if
          that is auto, treat as 0 or min-content per Flexbox rules".
        * As a maximum, 'auto' would mean "use the max-content size".
Comment 1 Sergio Villar Senin 2015-09-14 06:01:31 PDT 
This involves merging the following two patches from Blink:

* [CSSGridLayout] Use automatic minimums for 'auto' tracks r198697
* [CSS Grid Layout] The 'auto' value is valid for maxTrackBreadth property r200478
Comment 2 Sergio Villar Senin 2015-09-15 02:05:17 PDT 
Created attachment 261181 [details]
Patch
Comment 3 Darin Adler 2015-09-16 10:18:04 PDT 
Comment on attachment 261181 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=261181&action=review

> Source/WebCore/rendering/RenderGrid.cpp:559
> +        return LayoutUnit();

In new code we often use { } instead of LayoutUnit(); a bit more terse, not 100% sure it’s better.
Comment 4 Sergio Villar Senin 2015-09-17 05:13:53 PDT 
Committed r189911: <http://trac.webkit.org/changeset/189911>