[CSS Grid Layout] Tracks growing beyond limits when they should not

LayoutTests/fast/css-grid-layout/grid-content-sized-columns-resolution.html

Index: LayoutTests/fast/css-grid-layout/grid-content-sized-columns-resolution.html
diff --git a/LayoutTests/fast/css-grid-layout/grid-content-sized-columns-resolution.html b/LayoutTests/fast/css-grid-layout/grid-content-sized-columns-resolution.html
index 45ab233714e6e12db076050121a2635bc228a3dc..db1eb818a018ecf2127f1fe1a6820b43c3c1b65f 100644
--- a/LayoutTests/fast/css-grid-layout/grid-content-sized-columns-resolution.html
+++ b/LayoutTests/fast/css-grid-layout/grid-content-sized-columns-resolution.html
@@ -47,6 +47,12 @@
  .gridAutoAndFixedFixedAndMaxContentFixed {
      grid-template-columns: auto minmax(20px, 30px) minmax(max-content, 20px);
  }
+ .gridMaxContentAndMaxContentFixedAndMaxContent {
+    grid-template-columns: max-content minmax(max-content, 20px) max-content;
+ }
+ .gridAutoAndMinContentFixedAndMinContent {
+    grid-template-columns: auto minmax(min-content, 30px) min-content;
+ }
 
 </style>
 <script src="../../resources/js-test.js"></script>
@@ -295,6 +301,18 @@
     </div>
 </div>
 
+<div class="grid gridMaxContentAndMaxContentFixedAndMaxContent" id="gridMaxContentAndMaxContentFixedAndMaxContent">
+    <div style="grid-row: 1; grid-column: 1;">X X X</div>
+    <div style="grid-row: 1; grid-column: 3;">X X</div>
+    <div style="grid-row: 1; grid-column: 1 / -1;">XX XX XX XX XX</div>
+</div>
+
+<div class="grid gridAutoAndMinContentFixedAndMinContent" id="gridAutoAndMinContentFixedAndMinContent">
+    <div style="grid-row: 1; grid-column: 1;">XX</div>
+    <div style="grid-row: 1; grid-column: 1 / -1;">XXXXXXXXXXXXXXX</div>
+    <div style="grid-row: 1; grid-column: 3;">XXX XXX</div>
+</div>
+
 <script>
    function testGridColumnsValues(id, computedColumnValue)
    {
@@ -350,6 +368,8 @@
    testGridColumnsValues("gridMaxContentFixedAndAutoAboveLimits", "65px 85px");
    testGridColumnsValues("gridMinContentFixedAndFixedFixedAndAuto", "20px 20px 60px");
    testGridColumnsValues("gridAutoAndFixedFixedAndMaxContentFixed", "70px 20px 60px");
+   testGridColumnsValues("gridMaxContentAndMaxContentFixedAndMaxContent", "70px 20px 50px");

[Julien - ping for review, 2015/01/23 10:20:12]

This result doesn't seem correct. We should spread the extra space over all the
columns AFAICT.

I don't think this should block the patch but we should follow-up.

[svillar, 2015/01/23 10:39:47]

I am not that sure about that. Note that the second track has a fixed size
growth limit. This means that for growing beyond limits, that track won't be
eligible, because for max-content min track functions we should only grow those
tracks with max-content max track sizing functions.

+   testGridColumnsValues("gridAutoAndMinContentFixedAndMinContent", "55px 30px 65px");
 </script>
 </body>
 </html>