[Webkit-unassigned] [Bug 141422] New: [CSS Grid Layout] Update track sizes after distributing extra space

[bugzilla-daemon at webkit.org]

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

            Bug ID: 141422
           Summary: [CSS Grid Layout] Update track sizes after
                    distributing extra space
    Classification: Unclassified
           Product: WebKit
           Version: 528+ (Nightly build)
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: CSS
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: svillar at igalia.com

Both old and new tracks sizing algorithms instruct us to update the sizes of the content sized tracks only after distributing the extra space for all the items in the same span group. So far we have been doing it inside distributeSpaceToTracks(), i.e., once for every single item. That is wrong because it makes the algorithm order dependent.

The problem is that our code is like this:

resolveContentBasedTrackSizingFunctions()
    for each item resolveContentBasedTrackSizingFunctionsForItems() (x4)
        distributeSpaceToTracks()
            updateTrackSizes();

We must move the update out of the distributeSpaceToTracks(). We also need to call resolveContentBasedTrackSizingFunctionsForItems() with all the items with a particular span instead of one by one.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150210/ad964b02/attachment-0002.html>