[Webkit-unassigned] [Bug 123989] [meta][CSS Grid Layout] Speed up track layout

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Nov 7 08:27:36 PST 2013


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





--- Comment #3 from Sergio Villar Senin <svillar at igalia.com>  2013-11-07 08:26:22 PST ---
Some other potential merges:

    Remove an extra relayout for non-percent logical height grid items

    The only reason for a grid item to force a relayout after a grid
    area's logical height change is if it has a percentage height. In
    all other cases, this would cause an unneeded relayout. Note that
    changing the logical width still forces a relayout as the situation
    is not that simple.

    This improves auto-grid-lots-of-data.html from 94 runs/s to 226
    runs/s as we avoid a relayout in layoutGridItems for every grid
    items.

    BUG=273238

    Review URL: https://chromiumcodereview.appspot.com/23694023

and

Content-sized resolution should only happen on content-sized tracks

    The current code would always try to run the content sized
    algorithm regardless of whether this was needed. On top of
    it, it would iterate over ALL grid items, even if only a
    small set of the tracks were content-sized.

    This change stores of which tracks are content-sized and
    use the internal grid to only iterate over these tracks.

    On a Z620, this speeds up
    PerformanceTests/Layout/fixed-grid-lots-of-data.html from
    about 550 runs / sec to 830 runs / sec.

    BUG=273238

    Review URL: https://chromiumcodereview.appspot.com/22867006

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list