Changeset 200755 in webkit
- Timestamp:
- May 12, 2016 2:12:13 AM (8 years ago)
- Location:
- trunk
- Files:
-
- 12 edited
Legend:
- Unmodified
- Added
- Removed
-
trunk/LayoutTests/ChangeLog
r200746 r200755 1 2016-05-12 Manuel Rego Casasnovas <rego@igalia.com> 2 3 [css-grid] Incorrect parsing when using just 'span" as grid-line value 4 https://bugs.webkit.org/show_bug.cgi?id=157615 5 6 Reviewed by Sergio Villar Senin. 7 8 Add new test case in grid-item-column-row-get-set.html. 9 10 Updated other tests that were using "span" alone, with 2 approaches: 11 - In some of them changing "span" by "span 1" making it valid. 12 - In other changing the expected result to "auto", as it's invalid now. 13 14 * fast/css-grid-layout/grid-item-area-get-set.html: 15 * fast/css-grid-layout/grid-item-bad-named-area-auto-placement.html: 16 * fast/css-grid-layout/grid-item-column-row-get-set-expected.txt: 17 * fast/css-grid-layout/grid-item-column-row-get-set.html: 18 * fast/css-grid-layout/grid-item-end-after-get-set-expected.txt: 19 * fast/css-grid-layout/grid-item-end-after-get-set.html: 20 * fast/css-grid-layout/grid-item-negative-position-resolution.html: 21 * fast/css-grid-layout/grid-item-start-before-get-set-expected.txt: 22 * fast/css-grid-layout/grid-item-start-before-get-set.html: 23 1 24 2016-05-12 Joseph Pecoraro <pecoraro@apple.com> 2 25 -
trunk/LayoutTests/fast/css-grid-layout/grid-item-area-get-set.html
r200510 r200755 26 26 27 27 #threeValueGridArea { 28 grid-area: span / 10 / -1;28 grid-area: span 1 / 10 / -1; 29 29 } 30 30 … … 34 34 35 35 #fourValueMixedGridArea { 36 grid-area: firstArea / span / nonexistent / 1 foobar;36 grid-area: firstArea / span 1 / nonexistent / 1 foobar; 37 37 } 38 38 </style> -
trunk/LayoutTests/fast/css-grid-layout/grid-item-bad-named-area-auto-placement.html
r200510 r200755 32 32 .namedGridLineSpanColumn { 33 33 grid-row: 1; 34 grid-column: nonExistentArea / span ;34 grid-column: nonExistentArea / span 1; 35 35 background-color: lime; 36 36 } -
trunk/LayoutTests/fast/css-grid-layout/grid-item-column-row-get-set-expected.txt
r200510 r200755 53 53 PASS getComputedStyle(gridItemWithAfterSpanElement, '').getPropertyValue('grid-row-start') is "3" 54 54 PASS getComputedStyle(gridItemWithAfterSpanElement, '').getPropertyValue('grid-row-end') is "span 5" 55 PASS getComputedStyle(gridItemWith2OnlySpanElement, '').getPropertyValue('grid-column') is " span 1 / span 1"56 PASS getComputedStyle(gridItemWith2OnlySpanElement, '').getPropertyValue('grid-column-start') is " span 1"57 PASS getComputedStyle(gridItemWith2OnlySpanElement, '').getPropertyValue('grid-column-end') is " span 1"58 PASS getComputedStyle(gridItemWith2OnlySpanElement, '').getPropertyValue('grid-row') is " span 1 / span 1"59 PASS getComputedStyle(gridItemWith2OnlySpanElement, '').getPropertyValue('grid-row-start') is " span 1"60 PASS getComputedStyle(gridItemWith2OnlySpanElement, '').getPropertyValue('grid-row-end') is " span 1"55 PASS getComputedStyle(gridItemWith2OnlySpanElement, '').getPropertyValue('grid-column') is "auto / auto" 56 PASS getComputedStyle(gridItemWith2OnlySpanElement, '').getPropertyValue('grid-column-start') is "auto" 57 PASS getComputedStyle(gridItemWith2OnlySpanElement, '').getPropertyValue('grid-column-end') is "auto" 58 PASS getComputedStyle(gridItemWith2OnlySpanElement, '').getPropertyValue('grid-row') is "auto / auto" 59 PASS getComputedStyle(gridItemWith2OnlySpanElement, '').getPropertyValue('grid-row-start') is "auto" 60 PASS getComputedStyle(gridItemWith2OnlySpanElement, '').getPropertyValue('grid-row-end') is "auto" 61 61 PASS getComputedStyle(gridItemWith2AutoElement, '').getPropertyValue('grid-column') is "auto / auto" 62 62 PASS getComputedStyle(gridItemWith2AutoElement, '').getPropertyValue('grid-column-start') is "auto" … … 171 171 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-row-start') is "auto" 172 172 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-row-end') is "8" 173 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-column') is "auto / auto" 174 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-column-start') is "auto" 175 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-column-end') is "auto" 176 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-row') is "auto / auto" 177 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-row-start') is "auto" 178 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-row-end') is "auto" 173 179 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-column') is "span 1 / 3" 174 180 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-column-start') is "span 1" … … 461 467 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-row-start') is "auto" 462 468 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-row-end') is "auto" 469 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-column') is "auto / auto" 470 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-column-start') is "auto" 471 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-column-end') is "auto" 472 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-row') is "auto / auto" 473 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-row-start') is "auto" 474 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-row-end') is "auto" 475 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-column') is "auto / auto" 476 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-column-start') is "auto" 477 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-column-end') is "auto" 478 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-row') is "auto / auto" 479 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-row-start') is "auto" 480 PASS getComputedStyle(gridItem, '').getPropertyValue('grid-row-end') is "auto" 463 481 PASS successfullyParsed is true 464 482 -
trunk/LayoutTests/fast/css-grid-layout/grid-item-column-row-get-set.html
r200510 r200755 77 77 } 78 78 .gridItemWithNamedGridAreaAndSpan { 79 grid-column: thirdArea / span ;79 grid-column: thirdArea / span 1; 80 80 grid-row: firstArea / span 2; 81 81 } … … 117 117 testColumnRowCSSParsing("gridItemWithBeforeSpanElement", "span 2 / 4", "3 / span 5"); 118 118 testColumnRowCSSParsing("gridItemWithAfterSpanElement", "span 2 / 4", "3 / span 5"); 119 testColumnRowCSSParsing("gridItemWith2OnlySpanElement", " span 1 / span 1", "span 1 / span 1");119 testColumnRowCSSParsing("gridItemWith2OnlySpanElement", "auto / auto", "auto / auto"); 120 120 testColumnRowCSSParsing("gridItemWith2AutoElement", "auto / auto" , "auto / auto"); 121 121 testColumnRowCSSParsing("gridItemWithBothShortLongHandElement", "10 / 11", "4 / 5"); … … 149 149 testColumnRowJSParsing("4 / auto", "5 / auto"); 150 150 testColumnRowJSParsing("auto / 5", "auto / 8"); 151 testColumnRowJSParsing("span / 3", "5 / span", "span 1 / 3", "5 / span 1"); 151 testColumnRowJSParsing("span / 3", "5 / span", "auto / auto", "auto / auto"); 152 testColumnRowJSParsing("span 1 / 3", "5 / span 1", "span 1 / 3", "5 / span 1"); 152 153 testColumnRowJSParsing("first span / 3", "5 / last span", "span 1 first / 3", "5 / span 1 last"); 153 154 testColumnRowJSParsing("first / last", "nav / last span", "first / last", "nav / span 1 last"); … … 201 202 testColumnRowInvalidJSParsing("6 / 0", "8 / 0"); 202 203 testColumnRowInvalidJSParsing("0", "0"); 204 205 // 'span' alone is invalid 206 testColumnRowInvalidJSParsing("span", "span"); 207 testColumnRowInvalidJSParsing("span / span", "span / span"); 203 208 204 209 testColumnRowInvalidJSParsing("span span / span span", "span span / span span"); -
trunk/LayoutTests/fast/css-grid-layout/grid-item-end-after-get-set-expected.txt
r200510 r200755 35 35 PASS getComputedStyle(gridItemWithAfterSpan, '').getPropertyValue('grid-row-start') is "auto" 36 36 PASS getComputedStyle(gridItemWithAfterSpan, '').getPropertyValue('grid-row-end') is "span 9" 37 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-column') is "auto / span 1"37 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-column') is "auto / auto" 38 38 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-column-start') is "auto" 39 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-column-end') is " span 1"40 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-row') is "auto / span 1"39 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-column-end') is "auto" 40 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-row') is "auto / auto" 41 41 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-row-start') is "auto" 42 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-row-end') is " span 1"42 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-row-end') is "auto" 43 43 PASS getComputedStyle(gridItemWithAutoElement, '').getPropertyValue('grid-column') is "auto / auto" 44 44 PASS getComputedStyle(gridItemWithAutoElement, '').getPropertyValue('grid-column-start') is "auto" -
trunk/LayoutTests/fast/css-grid-layout/grid-item-end-after-get-set.html
r200510 r200755 82 82 testColumnRowCSSParsing("gridItemWithBeforeSpan", "auto / span 2", "auto / span 9"); 83 83 testColumnRowCSSParsing("gridItemWithAfterSpan", "auto / span 2", "auto / span 9"); 84 testColumnRowCSSParsing("gridItemWithOnlySpan", "auto / span 1", "auto / span 1");84 testColumnRowCSSParsing("gridItemWithOnlySpan", "auto / auto", "auto / auto"); 85 85 testColumnRowCSSParsing("gridItemWithAutoElement", "auto / auto", "auto / auto"); 86 86 testColumnRowCSSParsing("gridItemWithCustomIdent", "auto / first", "auto / last"); -
trunk/LayoutTests/fast/css-grid-layout/grid-item-negative-position-resolution.html
r200510 r200755 60 60 61 61 .negativeEndPositionStartSpanInColumnDirection { 62 grid-column: span / -1;62 grid-column: span 1 / -1; 63 63 grid-row: 1; 64 64 } … … 70 70 71 71 .negativeNamedGridLineEndPositionStartSpanInColumnDirection { 72 grid-column: span / last;72 grid-column: span 1 / last; 73 73 grid-row: 1; 74 74 } -
trunk/LayoutTests/fast/css-grid-layout/grid-item-start-before-get-set-expected.txt
r200510 r200755 33 33 PASS getComputedStyle(gridItemWithAfterSpan, '').getPropertyValue('grid-row-start') is "span 8" 34 34 PASS getComputedStyle(gridItemWithAfterSpan, '').getPropertyValue('grid-row-end') is "auto" 35 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-column') is " span 1/ auto"36 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-column-start') is " span 1"35 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-column') is "auto / auto" 36 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-column-start') is "auto" 37 37 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-column-end') is "auto" 38 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-row') is " span 1/ auto"39 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-row-start') is " span 1"38 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-row') is "auto / auto" 39 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-row-start') is "auto" 40 40 PASS getComputedStyle(gridItemWithOnlySpan, '').getPropertyValue('grid-row-end') is "auto" 41 41 PASS getComputedStyle(gridItemWithAutoElement, '').getPropertyValue('grid-column') is "auto / auto" -
trunk/LayoutTests/fast/css-grid-layout/grid-item-start-before-get-set.html
r200510 r200755 87 87 testColumnRowCSSParsing("gridItemWithBeforeSpan", "span 2 / auto", "span 8 / auto"); 88 88 testColumnRowCSSParsing("gridItemWithAfterSpan", "span 2 / auto", "span 8 / auto"); 89 testColumnRowCSSParsing("gridItemWithOnlySpan", " span 1 / auto", "span 1/ auto");89 testColumnRowCSSParsing("gridItemWithOnlySpan", "auto / auto", "auto / auto"); 90 90 testColumnRowCSSParsing("gridItemWithAutoElement", "auto / auto", "auto / auto"); 91 91 testColumnRowCSSParsing("gridItemWithCustomIdent", "first / auto", "last / auto"); -
trunk/Source/WebCore/ChangeLog
r200747 r200755 1 2016-05-12 Manuel Rego Casasnovas <rego@igalia.com> 2 3 [css-grid] Incorrect parsing when using just 'span" as grid-line value 4 https://bugs.webkit.org/show_bug.cgi?id=157615 5 6 Reviewed by Sergio Villar Senin. 7 8 The spec is pretty clear about this: 9 https://drafts.csswg.org/css-grid/#typedef-grid-row-start-grid-line 10 11 The possible options when using "span" to reference a grid line are: 12 [ span && [ <integer> || <custom-ident> ] ] 13 14 Thus, "span" keyword alone produces an invalid declaration. 15 16 * css/CSSParser.cpp: 17 (WebCore::CSSParser::parseGridPosition): 18 1 19 2016-05-12 Csaba Osztrogonác <ossy@webkit.org> 2 20 -
trunk/Source/WebCore/css/CSSParser.cpp
r200626 r200755 5502 5502 // If we didn't parse anything, this is not a valid grid position. 5503 5503 if (!hasSeenSpanKeyword && !gridLineName && !numericValue) 5504 return nullptr; 5505 5506 // If we have "span" keyword alone is invalid. 5507 if (hasSeenSpanKeyword && !gridLineName && !numericValue) 5504 5508 return nullptr; 5505 5509
Note: See TracChangeset
for help on using the changeset viewer.