For flex items, percent paddings should resolve against their respective dimension

Source/core/rendering/RenderBoxModelObject.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  *           (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 292 matching lines @@
 int RenderBoxModelObject::pixelSnappedOffsetWidth() const
 {
     return snapSizeToPixel(offsetWidth(), offsetLeft());
 }
 
 int RenderBoxModelObject::pixelSnappedOffsetHeight() const
 {
     return snapSizeToPixel(offsetHeight(), offsetTop());
 }
 
-LayoutUnit RenderBoxModelObject::computedCSSPadding(const Length& padding) const
+LayoutUnit RenderBoxModelObject::computedCSSPadding(const Length& padding, PaddingType type) const
 {
     LayoutUnit w = 0;
-    if (padding.isPercent())
-        w = containingBlockLogicalWidthForContent();
+    if (padding.isPercent()) {
+        if (containingBlock()->isFlexibleBox() && RuntimeEnabledFeatures::verticalPaddingEnabled()) {
+            if (type == TopPadding || type == BottomPadding || type == BeforePadding || type == AfterPadding)
+                w = isHorizontalWritingMode() ? containingBlockLogicalHeightForPadding() : containingBlockLogicalWidthForContent();
+            else
+                w = isHorizontalWritingMode() ? containingBlockLogicalWidthForContent() : containingBlockLogicalHeightForPadding();
+        } else {
+            w = containingBlockLogicalWidthForContent();
+        }
+    }
     return minimumValueForLength(padding, w);
 }
 
 RoundedRect RenderBoxModelObject::getBackgroundRoundedRect(const LayoutRect& borderRect, InlineFlowBox* box, LayoutUnit inlineBoxWidth, LayoutUnit inlineBoxHeight,
     bool includeLogicalLeftEdge, bool includeLogicalRightEdge) const
 {
     RoundedRect border = style()->getRoundedBorderFor(borderRect, includeLogicalLeftEdge, includeLogicalRightEdge);
     if (box && (box->nextLineBox() || box->prevLineBox())) {
         RoundedRect segmentBorder = style()->getRoundedBorderFor(LayoutRect(0, 0, inlineBoxWidth, inlineBoxHeight), includeLogicalLeftEdge, includeLogicalRightEdge);
         border.setRadii(segmentBorder.radii());
@@ skipping 2172 matching lines @@
             context->drawInnerShadow(border, shadowColor, flooredIntSize(shadowOffset), shadowBlur, shadowSpread, clippedEdges);
         }
     }
 }
 
 LayoutUnit RenderBoxModelObject::containingBlockLogicalWidthForContent() const
 {
     return containingBlock()->availableLogicalWidth();
 }
 
+LayoutUnit RenderBoxModelObject::containingBlockLogicalHeightForPadding() const
+{
+    switch (containingBlock()->style()->logicalHeight().type()) {
+    case Auto:
+    case MinContent:
+    case MaxContent:
+    case FitContent:
+    case FillAvailable:
+        return 0;
+    case Fixed:
+    case Calculated:
+        return containingBlock()->style()->logicalHeight().value();
+    case Percent:
+        return computePercentLogicalHeightForPadding(containingBlock());
+    default:

[tony, 2014/07/16 16:57:23]

One of the reasons to use a switch statement is so you can enumerate all the
values and get a compiler error if someone adds a new type.  Please replace the
|default| label with the other values and add an ASSERT_NOT_REACHED() if
appropriate-- you'll need to see if the other LengthTypes can be set to a height
or width.

[harpreet.sk, 2014/07/17 14:36:53]

Done.

+        return 0;
+    }
+}
+
+LayoutUnit RenderBoxModelObject::computePercentLogicalHeightForPadding(RenderBlock* block) const
+{
+    ASSERT(block->style()->logicalHeight().isPercent());
+
+    float percentHeightFactor = block->style()->logicalHeight().value() / 100;
+    RenderBlock* cb = block->containingBlock();
+    while (cb) {
+        if (cb->style()->logicalHeight().isIntrinsicOrAuto())
+            return 0;
+        if (!block->skipContainingBlockForPercentHeightCalculation(cb) && (cb->style()->logicalHeight().isFixed() || cb->style()->logicalHeight().isCalculated()))
+            break;
+        if (cb->style()->logicalHeight().isPercent())
+            percentHeightFactor *= cb->style()->logicalHeight().value() / 100;
+        cb = cb->containingBlock();
+    }
+
+    if (cb)
+        return percentHeightFactor * cb->style()->logicalHeight().value();
+
+    return 0;
+}
+
 RenderBoxModelObject* RenderBoxModelObject::continuation() const
 {
     if (!continuationMap)
         return 0;
     return continuationMap->get(this);
 }
 
 void RenderBoxModelObject::setContinuation(RenderBoxModelObject* continuation)
 {
     if (continuation) {
@@ skipping 207 matching lines @@
     ASSERT(!beforeChild || toBoxModelObject == beforeChild->parent());
     for (RenderObject* child = startChild; child && child != endChild; ) {
         // Save our next sibling as moveChildTo will clear it.
         RenderObject* nextSibling = child->nextSibling();
         moveChildTo(toBoxModelObject, child, beforeChild, fullRemoveInsert);
         child = nextSibling;
     }
 }
 
 } // namespace WebCore