Changeset 183739 in webkit
- Timestamp:
- May 4, 2015 1:37:03 AM (9 years ago)
- Location:
- trunk
- Files:
-
- 2 added
- 3 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/LayoutTests/ChangeLog
r183732 r183739 1 2015-05-04 Manuel Rego Casasnovas <rego@igalia.com> 2 3 [CSS Grid Layout] Wrong computed style for named grid lines in implicit tracks 4 https://bugs.webkit.org/show_bug.cgi?id=144449 5 6 Reviewed by Sergio Villar Senin. 7 8 * fast/css-grid-layout/named-grid-lines-computed-style-implicit-tracks-expected.txt: Added. 9 * fast/css-grid-layout/named-grid-lines-computed-style-implicit-tracks.html: Added. 10 1 11 2015-05-02 Simon Fraser <simon.fraser@apple.com> 2 12 -
trunk/Source/WebCore/ChangeLog
r183735 r183739 1 2015-05-04 Manuel Rego Casasnovas <rego@igalia.com> 2 3 [CSS Grid Layout] Wrong computed style for named grid lines in implicit tracks 4 https://bugs.webkit.org/show_bug.cgi?id=144449 5 6 Reviewed by Sergio Villar Senin. 7 8 If there're implicit tracks then trackPositions is bigger than 9 trackSizes, so we need to use the proper index to append the trailing 10 <ident>s in valueForGridTrackList(). 11 12 Test: fast/css-grid-layout/named-grid-lines-computed-style-implicit-tracks.html 13 14 * css/CSSComputedStyleDeclaration.cpp: 15 (WebCore::valueForGridTrackList): 16 1 17 2015-05-03 Gyuyoung Kim <gyuyoung.kim@webkit.org> 2 18 -
trunk/Source/WebCore/css/CSSComputedStyleDeclaration.cpp
r183591 r183739 1054 1054 1055 1055 auto list = CSSValueList::createSpaceSeparated(); 1056 unsigned insertionIndex; 1056 1057 if (isRenderGrid) { 1057 1058 const Vector<LayoutUnit>& trackPositions = direction == ForColumns ? downcast<RenderGrid>(*renderer).columnPositions() : downcast<RenderGrid>(*renderer).rowPositions(); … … 1064 1065 list.get().append(zoomAdjustedPixelValue(trackPositions[i + 1] - trackPositions[i], style)); 1065 1066 } 1067 insertionIndex = trackPositions.size() - 1; 1066 1068 } else { 1067 1069 for (unsigned i = 0; i < trackSizes.size(); ++i) { … … 1069 1071 list.get().append(specifiedValueForGridTrackSize(trackSizes[i], style)); 1070 1072 } 1073 insertionIndex = trackSizes.size(); 1071 1074 } 1072 1075 1073 1076 // Those are the trailing <ident>* allowed in the syntax. 1074 addValuesForNamedGridLinesAtIndex(orderedNamedGridLines, trackSizes.size(), list.get());1077 addValuesForNamedGridLinesAtIndex(orderedNamedGridLines, insertionIndex, list.get()); 1075 1078 return WTF::move(list); 1076 1079 }
Note: See TracChangeset
for help on using the changeset viewer.