[css-grid] Clearing the override height before layout

third_party/WebKit/Source/core/layout/LayoutGrid.cpp

 /*
  * Copyright (C) 2011 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 217 matching lines @@
 
   {
     // LayoutState needs this deliberate scope to pop before updating scroll
     // information (which may trigger relayout).
     LayoutState state(*this);
 
     LayoutSize previous_size = Size();
 
     // We need to clear both own and containingBlock override sizes to
     // ensure we get the same result when grid's intrinsic size is
     // computed again in the updateLogicalWidth call bellow.

[Manuel Rego, 2017/04/21 05:21:11]

This comment needs to be updated now.

[jfernandez, 2017/04/21 08:31:47]

Well, the comment still applies, IMO. It's just that we clear the item's
overrideSize in any case. However, in the case of content-sized
grid container we still need to cleare the containingBlockOverride size and
force a layout, as the comment states.

[Manuel Rego, 2017/04/21 08:51:51]

But the comment says:
// We need to clear both **own** and containingBlock override sizes

Before we clear both if that condition applies, now we clear it always.
That's why I say the comment needs to be updated.

-    if (SizesLogicalWidthToFitContent(StyleRef().LogicalWidth()) ||
-        StyleRef().LogicalWidth().IsIntrinsicOrAuto()) {
-      for (auto* child = FirstInFlowChildBox(); child;
-           child = child->NextInFlowSiblingBox()) {
+    for (auto* child = FirstInFlowChildBox(); child;
+         child = child->NextInFlowSiblingBox()) {
+      child->ClearOverrideSize();
+      if (SizesLogicalWidthToFitContent(StyleRef().LogicalWidth()) ||
+          StyleRef().LogicalWidth().IsIntrinsicOrAuto()) {
         if (!IsOrthogonalChild(*child))
           continue;
-        child->ClearOverrideSize();
         child->ClearContainingBlockOverrideSize();
         child->ForceLayout();
       }
     }
 
     UpdateLogicalWidth();
     has_definite_logical_height_ = HasDefiniteLogicalHeight();
 
     TextAutosizer::LayoutScope text_autosizer_layout_scope(this, &layout_scope);
 
@@ skipping 1264 matching lines @@
     const LayoutBox& child,
     GridTrackSizingDirection direction) const {
   return !IsOrthogonalChild(child)
              ? direction
              : (direction == kForColumns ? kForRows : kForColumns);
 }
 
 // FIXME: This logic is shared by LayoutFlexibleBox, so it should be moved to
 // LayoutBox.
 void LayoutGrid::ApplyStretchAlignmentToChildIfNeeded(LayoutBox& child) {
-  // We clear height override values because we will decide now whether it's
-  // allowed or not, evaluating the conditions which might have changed since
-  // the old values were set.
-  child.ClearOverrideLogicalContentHeight();
-
   GridTrackSizingDirection child_block_direction =
       FlowAwareDirectionForChild(child, kForRows);
   bool block_flow_is_column_axis = child_block_direction == kForRows;
   bool allowed_to_stretch_child_block_size =
       block_flow_is_column_axis ? AllowedToStretchChildAlongColumnAxis(child)
                                 : AllowedToStretchChildAlongRowAxis(child);
   if (allowed_to_stretch_child_block_size) {
     LayoutUnit stretched_logical_height =
         AvailableAlignmentSpaceForChildBeforeStretching(
             OverrideContainingBlockContentSizeForChild(child,
@@ skipping 812 matching lines @@
   if (direction == kForRows)
     return grid.NumTracks(kForRows);
 
   return grid.NumTracks(kForRows)
              ? grid.NumTracks(kForColumns)
              : GridPositionsResolver::ExplicitGridColumnCount(
                    StyleRef(), grid.AutoRepeatTracks(kForColumns));
 }
 
 }  // namespace blink