Blink doesn't honor percent heights on children of "align-self:stretch" flex items in a fixed-height

Source/core/rendering/RenderFlexibleBox.cpp

Index: Source/core/rendering/RenderFlexibleBox.cpp
diff --git a/Source/core/rendering/RenderFlexibleBox.cpp b/Source/core/rendering/RenderFlexibleBox.cpp
index 6114d5654cd0fe113c50d191fd0a018597dff66b..ad21190b7b537ab08ee6b387b78d2fefa7c1c949 100644
--- a/Source/core/rendering/RenderFlexibleBox.cpp
+++ b/Source/core/rendering/RenderFlexibleBox.cpp
@@ -1375,6 +1375,7 @@ void RenderFlexibleBox::applyStretchAlignmentToChild(RenderBox* child, LayoutUni
             // FIXME: Can avoid laying out here in some cases. See https://webkit.org/b/87905.
             if (desiredLogicalHeight != child->logicalHeight()) {
                 child->setOverrideLogicalContentHeight(desiredLogicalHeight - child->borderAndPaddingLogicalHeight());
+                child->setOverrideContainingBlockContentLogicalHeight(stretchedLogicalHeight - child->borderAndPaddingLogicalHeight());

[cbiesinger, 2014/06/24 03:01:28]

I wrote this comment below before I thought of the suggestion I gave above. I
decided to keep it here in case it is useful for future reference.

---

So, I see in the comments that you changed this from desiredLogicalHeight, but
this new code is wrong. It has to be desiredLogicalHeight, otherwise the
percentage height of the children of the flex item will be incorrect if there's
a max-height on the flex item.

Irrespective of that you do need to clear this width so that the code will work
correctly if the align-self property gets changed from stretch to something
else. Tony's example should be fine because when we mark a node as needing
layout we also mark its ancestors as needing layout, but you should add a test
to make sure.

[harpreet.sk, 2014/06/24 07:57:00]

I had made the changes to add desiredLogicalHeight instead of
stretchedLogicalHeight and clearing the 
overrideContainingBlockContentLogicalHeight afterwards. 
The case tony mention of updating height of grandchild to different percent
value is already there in layouttest.

                 child->setLogicalHeight(0);
                 child->forceChildLayout();
             }