Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(163)

Unified Diff: LayoutTests/fast/css-grid-layout/grid-item-removal-auto-placement-update.html

Issue 208133003: [CSS Grid Layout] Prevent issues with checkLayout() in grid items (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 9 months 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
Index: LayoutTests/fast/css-grid-layout/grid-item-removal-auto-placement-update.html
diff --git a/LayoutTests/fast/css-grid-layout/grid-item-removal-auto-placement-update.html b/LayoutTests/fast/css-grid-layout/grid-item-removal-auto-placement-update.html
index 23a4b5313f44e7bef7c3778d07884328de26e9b6..5e7e4f21c1ea4115628995b29e36011dd4130149 100644
--- a/LayoutTests/fast/css-grid-layout/grid-item-removal-auto-placement-update.html
+++ b/LayoutTests/fast/css-grid-layout/grid-item-removal-auto-placement-update.html
@@ -20,30 +20,31 @@ function testRemoval(gridElementID, autoFlowElementID, size)
autoFlowElement.setAttribute("data-expected-width", size.width);
autoFlowElement.setAttribute("data-expected-height", size.height);
- checkLayout("#" + gridElementID);
+ checkLayout("#" + gridElementID, document.getElementById("test-output"));
}
function testRemovals()
{
- checkLayout("#autoFlowColumnElement");
+ var testOutput = document.getElementById("test-output");
+ checkLayout("#autoFlowColumnElement", testOutput);
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("#autoFlowRowElement", testOutput);
testRemoval("gridAutoFlowRow", "autoFlowRowElement", { 'width': '100', 'height': '100' });
testRemoval("gridAutoFlowRow", "autoFlowRowElement", { 'width': '100', 'height': '50' });
testRemoval("gridAutoFlowRow", "autoFlowRowElement", { 'width': '50', 'height': '50' });
testRemoval("gridAutoFlowRow", "autoFlowRowElement", { 'width': '50', 'height': '50' });
- checkLayout("#gridAutoFlowColumnWithAutoItems");
+ checkLayout("#gridAutoFlowColumnWithAutoItems", testOutput);
testRemoval("gridAutoFlowColumnWithAutoItems", "autoFlowRowElementWithAutoItems", { 'width': '100', 'height': '100' });
testRemoval("gridAutoFlowColumnWithAutoItems", "autoFlowRowElementWithAutoItems", { 'width': '100', 'height': '50' });
testRemoval("gridAutoFlowColumnWithAutoItems", "autoFlowRowElementWithAutoItems", { 'width': '50', 'height': '100' });
testRemoval("gridAutoFlowColumnWithAutoItems", "autoFlowRowElementWithAutoItems", { 'width': '50', 'height': '50' });
- checkLayout("#gridAutoFlowRowWithAutoAndFixedItems");
+ checkLayout("#gridAutoFlowRowWithAutoAndFixedItems", testOutput);
testRemoval("gridAutoFlowRowWithAutoAndFixedItems", "autoFlowRowElementWithAutoAndFixedItems", { 'width': '100', 'height': '100' });
testRemoval("gridAutoFlowRowWithAutoAndFixedItems", "autoFlowRowElementWithAutoAndFixedItems", { 'width': '100', 'height': '50' });
testRemoval("gridAutoFlowRowWithAutoAndFixedItems", "autoFlowRowElementWithAutoAndFixedItems", { 'width': '100', 'height': '50' });
@@ -95,5 +96,7 @@ window.addEventListener("load", testRemovals, false);
</div>
</div>
+<div id="test-output"></div>
+
</body>
</html>

Powered by Google App Engine
This is Rietveld 408576698