<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>120495</bug_id>
          
          <creation_ts>2013-08-29 15:04:22 -0700</creation_ts>
          <short_desc>[Grid Layout] Content-sized resolution should only happen on content-sized tracks</short_desc>
          <delta_ts>2015-01-26 06:59:25 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Layout and Rendering</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>BlinkMergeCandidate</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Ryosuke Niwa">rniwa</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bdakin</cc>
    
    <cc>dino</cc>
    
    <cc>hyatt</cc>
    
    <cc>simon.fraser</cc>
    
    <cc>svillar</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>922685</commentid>
    <comment_count>0</comment_count>
    <who name="Ryosuke Niwa">rniwa</who>
    <bug_when>2013-08-29 15:04:22 -0700</bug_when>
    <thetext>Consider merging https://chromium.googlesource.com/chromium/blink/+/26c7fde92a23277ca9f38e605f41927812a9dc37

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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1064156</commentid>
    <comment_count>1</comment_count>
    <who name="Sergio Villar Senin">svillar</who>
    <bug_when>2015-01-26 06:59:25 -0800</bug_when>
    <thetext>This was fixed long time ago.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>