[css-grid] "normal" alignment is "start" for replaced elements

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 1400 matching lines @@
   // values. Hence, we need to compute margins in order to determine the
   // available height before stretching.
   return gridAreaBreadthForChild -
          (child.needsLayout()
               ? computeMarginLogicalSizeForChild(BlockDirection, child)
               : marginLogicalHeightForChild(child));
 }
 
 StyleSelfAlignmentData LayoutGrid::alignSelfForChild(
     const LayoutBox& child) const {
-  if (!child.isAnonymous())
-    return child.styleRef().resolvedAlignSelf(selfAlignmentNormalBehavior());
+  if (!child.isAnonymous()) {

[jfernandez, 2017/03/01 12:46:49]

Perhaps it'd be a better idea to solve directly the normal behavior instead of
using selfAlignmentNormalBehavior().

+    return child.styleRef().resolvedAlignSelf(
+        selfAlignmentNormalBehavior(&child));
+  }
   // All the 'auto' values has been solved by the StyleAdjuster, but it's
   // possible that some grid items generate Anonymous boxes, which need to be
   // solved during layout.
-  return child.styleRef().resolvedAlignSelf(selfAlignmentNormalBehavior(),
+  return child.styleRef().resolvedAlignSelf(selfAlignmentNormalBehavior(&child),
                                             style());
 }
 
 StyleSelfAlignmentData LayoutGrid::justifySelfForChild(
     const LayoutBox& child) const {
-  if (!child.isAnonymous())
-    return child.styleRef().resolvedJustifySelf(ItemPositionStretch);
+  if (!child.isAnonymous()) {

[jfernandez, 2017/03/01 12:46:49]

ditto.

+    return child.styleRef().resolvedJustifySelf(
+        selfAlignmentNormalBehavior(&child));
+  }
   // All the 'auto' values has been solved by the StyleAdjuster, but it's
   // possible that some grid items generate Anonymous boxes, which need to be
   // solved during layout.
-  return child.styleRef().resolvedJustifySelf(selfAlignmentNormalBehavior(),
-                                              style());
+  return child.styleRef().resolvedJustifySelf(
+      selfAlignmentNormalBehavior(&child), style());
 }
 
 GridTrackSizingDirection LayoutGrid::flowAwareDirectionForChild(
     const LayoutBox& child,
     GridTrackSizingDirection direction) const {
   return !isOrthogonalChild(child)
              ? direction
              : (direction == ForColumns ? ForRows : ForColumns);
 }
 
@@ skipping 642 matching lines @@
   if (direction == ForRows)
     return grid.numTracks(ForRows);
 
   return grid.numTracks(ForRows)
              ? grid.numTracks(ForColumns)
              : GridPositionsResolver::explicitGridColumnCount(
                    styleRef(), grid.autoRepeatTracks(ForColumns));
 }
 
 }  // namespace blink