LayoutTests/fast/css-grid-layout/resources/grid-columns-rows-get-set-multiple.js - Issue 23528004: [CSS Grid Layout] Update named grid lines syntax to the last version of the specs

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: LayoutTests/fast/css-grid-layout/resources/grid-columns-rows-get-set-multiple.js

Issue 23528004: [CSS Grid Layout] Update named grid lines syntax to the last version of the specs (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@named

Patch Set: New parsing based on rune's and jchaffraix's comments Created 7 years, 1 month ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

« LayoutTests/fast/css-grid-layout/named-grid-line-get-set.html ('K') | « LayoutTests/fast/css-grid-layout/resources/grid-columns-rows-get-set.js ('k') | LayoutTests/fast/css-grid-layout/resources/non-grid-columns-rows-get-set.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: LayoutTests/fast/css-grid-layout/resources/grid-columns-rows-get-set-multiple.js

diff --git a/LayoutTests/fast/css-grid-layout/resources/grid-columns-rows-get-set-multiple.js b/LayoutTests/fast/css-grid-layout/resources/grid-columns-rows-get-set-multiple.js

index 806226feea8ee3a89393a2c90827d2f796d2aadb..780bf31944c4212e57eb74696eef191247e9308f 100755

--- a/LayoutTests/fast/css-grid-layout/resources/grid-columns-rows-get-set-multiple.js

+++ b/LayoutTests/fast/css-grid-layout/resources/grid-columns-rows-get-set-multiple.js

@@ -63,8 +63,8 @@ function testInherit()

parentElement.style.width = "800px";

parentElement.style.height = "600px";

parentElement.style.font = "10px Ahem"; // Used to resolve em font consistently.

- parentElement.style.gridDefinitionColumns = "50px 1fr 'last'";

- parentElement.style.gridDefinitionRows = "2em 'middle' 45px";

+ parentElement.style.gridDefinitionColumns = "50px 1fr (last)";

+ parentElement.style.gridDefinitionRows = "2em (middle) 45px";

testGridDefinitionsValues(parentElement, "50px 750px last", "20px middle 45px");

element = document.createElement("div");

@@ -87,8 +87,8 @@ function testInitial()

element.style.display = "grid";

element.style.width = "800px";

element.style.height = "600px";

- element.style.gridDefinitionColumns = "150% 'middle' 55px";

- element.style.gridDefinitionRows = "1fr 'line' 2fr 'line'";

+ element.style.gridDefinitionColumns = "150% (middle) 55px";

+ element.style.gridDefinitionRows = "1fr (line) 2fr (line)";

testGridDefinitionsValues(element, "1200px middle 55px", "200px line 400px line");

element.style.gridDefinitionColumns = "initial";