[CSS Grid Layout] Prevent issues with checkLayout() in grid items
authorrego@igalia.com <rego@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 26 Mar 2014 11:11:53 +0000 (11:11 +0000)
committerrego@igalia.com <rego@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 26 Mar 2014 11:11:53 +0000 (11:11 +0000)
https://bugs.webkit.org/show_bug.cgi?id=130770

Reviewed by Sergio Villar Senin.

When we use checkLayout() on a grid item, the results (PASS or FAIL
messages) are added as new auto-placed items in the grid. Which might
cause some unexpected behavior in the tests.

This patch modifies the related layout tests to use a different
container (usually the parent node of the grid) for the test results.
Avoiding any kind of interference with the grid code.

* fast/css-grid-layout/grid-item-addition-auto-placement-update.html:
  Use parent node of grid when checking grid items. Check layout of the
  whole grid instead of grid item.
* fast/css-grid-layout/grid-item-removal-auto-placement-update.html:
  Check layout of the whole grid instead of grid item.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@166290 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/fast/css-grid-layout/grid-item-addition-auto-placement-update.html
LayoutTests/fast/css-grid-layout/grid-item-removal-auto-placement-update.html

index bdaf9d8..813cbf6 100644 (file)
@@ -1,3 +1,24 @@
+2014-03-26  Manuel Rego Casasnovas  <rego@igalia.com>
+
+        [CSS Grid Layout] Prevent issues with checkLayout() in grid items
+        https://bugs.webkit.org/show_bug.cgi?id=130770
+
+        Reviewed by Sergio Villar Senin.
+
+        When we use checkLayout() on a grid item, the results (PASS or FAIL
+        messages) are added as new auto-placed items in the grid. Which might
+        cause some unexpected behavior in the tests.
+
+        This patch modifies the related layout tests to use a different
+        container (usually the parent node of the grid) for the test results.
+        Avoiding any kind of interference with the grid code.
+
+        * fast/css-grid-layout/grid-item-addition-auto-placement-update.html:
+          Use parent node of grid when checking grid items. Check layout of the
+          whole grid instead of grid item.
+        * fast/css-grid-layout/grid-item-removal-auto-placement-update.html:
+          Check layout of the whole grid instead of grid item.
+
 2014-03-25  Jer Noble  <jer.noble@apple.com>
 
         [MSE] Duplicate 'seeked' events.
index 39ccfec..0e7861b 100644 (file)
@@ -25,18 +25,18 @@ function testAddition(gridElementID, position, autoFlowElementID, size)
     var autoFlowElement = document.getElementById(autoFlowElementID);
     autoFlowElement.setAttribute("data-expected-width", size.width);
     autoFlowElement.setAttribute("data-expected-height", size.height);
-    checkLayout("#" + autoFlowElementID);
+    checkLayout("#" + autoFlowElementID, gridElement.parentNode);
 }
 
 function testAdditions()
 {
-    checkLayout("#autoFlowColumnElement");
+    checkLayout("#gridAutoFlowColumn");
     testAddition("gridAutoFlowColumn", { 'row': '1', 'column': '1' }, "autoFlowColumnElement", { 'width': '50', 'height': '100' });
     testAddition("gridAutoFlowColumn", { 'row': '1', 'column': '2' }, "autoFlowColumnElement", { 'width': '50', 'height': '100' });
     testAddition("gridAutoFlowColumn", { 'row': '2', 'column': '2' }, "autoFlowColumnElement", { 'width': '50', 'height': '100' });
     testAddition("gridAutoFlowColumn", { 'row': '2', 'column': '1' }, "autoFlowColumnElement", { 'width': '170', 'height': '50' });
 
-    checkLayout("#autoFlowRowElement");
+    checkLayout("#gridAutoFlowRow");
     testAddition("gridAutoFlowRow", { 'row': '1', 'column': '1' }, "autoFlowRowElement", { 'width': '100', 'height': '50' });
     testAddition("gridAutoFlowRow", { 'row': '2', 'column': '1' }, "autoFlowRowElement", { 'width': '100', 'height': '50' });
     testAddition("gridAutoFlowRow", { 'row': '2', 'column': '2' }, "autoFlowRowElement", { 'width': '100', 'height': '50' });
index 859307f..bcbacae 100644 (file)
@@ -29,13 +29,13 @@ function testRemoval(gridElementID, autoFlowElementID, size)
 
 function testRemovals()
 {
-    checkLayout("#autoFlowColumnElement");
+    checkLayout("#gridAutoFlowColumn");
     testRemoval("gridAutoFlowColumn", "autoFlowColumnElement", { 'width': '100', 'height': '100' });
     testRemoval("gridAutoFlowColumn", "autoFlowColumnElement", { 'width': '100', 'height': '50' });
     testRemoval("gridAutoFlowColumn", "autoFlowColumnElement", { 'width': '50', 'height': '100' });
     testRemoval("gridAutoFlowColumn", "autoFlowColumnElement", { 'width': '50', 'height': '50' });
 
-    checkLayout("#autoFlowRowElement");
+    checkLayout("#gridAutoFlowRow");
     testRemoval("gridAutoFlowRow", "autoFlowRowElement", { 'width': '100', 'height': '100' });
     testRemoval("gridAutoFlowRow", "autoFlowRowElement", { 'width': '100', 'height': '50' });
     testRemoval("gridAutoFlowRow", "autoFlowRowElement", { 'width': '50', 'height': '50' });