Ignore:
Timestamp:
Oct 28, 2013, 7:58:43 PM (12 years ago)
Author:
[email protected]
Message:

RenderElement::style() should return a reference.
<​https://webkit.org/b/123414>

Now that renderers always have style, go ahead and make style()
return a RenderStyle&.

There are countless opportunities for further cleanup enabled by
this change. I'm simply passing &style() in many cases where we
can really do something nicer instead.

Reviewed by Anders Carlsson.

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/Source/WebCore/rendering/RenderDeprecatedFlexibleBox.cpp

    r158097 r158163  
    4444        , m_largestOrdinal(1)
    4545    {
    46         if (m_box->style()->boxOrient() == HORIZONTAL && !m_box->style()->isLeftToRightDirection())
    47             m_forward = m_box->style()->boxDirection() != BNORMAL;
     46        if (m_box->style().boxOrient() == HORIZONTAL && !m_box->style().isLeftToRightDirection())
     47            m_forward = m_box->style().boxDirection() != BNORMAL;
    4848        else
    49             m_forward = m_box->style()->boxDirection() == BNORMAL;
     49            m_forward = m_box->style().boxDirection() == BNORMAL;
    5050        if (!m_forward) {
    5151            // No choice, since we're going backwards, we have to find out the highest ordinal up front.
    5252            RenderBox* child = m_box->firstChildBox();
    5353            while (child) {
    54                 if (child->style()->boxOrdinalGroup() > m_largestOrdinal)
    55                     m_largestOrdinal = child->style()->boxOrdinalGroup();
     54                if (child->style().boxOrdinalGroup() > m_largestOrdinal)
     55                    m_largestOrdinal = child->style().boxOrdinalGroup();
    5656                child = child->nextSiblingBox();
    5757            }
    … …  
    9898
    9999            if (m_currentChild && notFirstOrdinalValue())
    100                 m_ordinalValues.add(m_currentChild->style()->boxOrdinalGroup());
     100                m_ordinalValues.add(m_currentChild->style().boxOrdinalGroup());
    101101        } while (!m_currentChild || (!m_currentChild->isAnonymous()
    102                  && m_currentChild->style()->boxOrdinalGroup() != m_currentOrdinal));
     102                 && m_currentChild->style().boxOrdinalGroup() != m_currentOrdinal));
    103103        return m_currentChild;
    104104    }
    … …  
    108108    {
    109109        unsigned int firstOrdinalValue = m_forward ? 1 : m_largestOrdinal;
    110         return m_currentOrdinal == firstOrdinalValue && m_currentChild->style()->boxOrdinalGroup() != firstOrdinalValue;
     110        return m_currentOrdinal == firstOrdinalValue && m_currentChild->style().boxOrdinalGroup() != firstOrdinalValue;
    111111    }
    112112
    … …  
    139139    // Auto and percentage margins simply become 0 when computing min/max width.
    140140    // Fixed margins can be added in as is.
    141     Length marginLeft = child->style()->marginLeft();
    142     Length marginRight = child->style()->marginRight();
     141    Length marginLeft = child->style().marginLeft();
     142    Length marginRight = child->style().marginRight();
    143143    LayoutUnit margin = 0;
    144144    if (marginLeft.isFixed())
    … …  
    152152{
    153153    // Positioned children and collapsed children don't affect the min/max width.
    154     return child->isOutOfFlowPositioned() || child->style()->visibility() == COLLAPSE;
     154    return child->isOutOfFlowPositioned() || child->style().visibility() == COLLAPSE;
    155155}
    156156
    … …  
    171171void RenderDeprecatedFlexibleBox::styleWillChange(StyleDifference diff, const RenderStyle& newStyle)
    172172{
    173     RenderStyle* oldStyle = hasInitializedStyle() ? style() : nullptr;
     173    RenderStyle* oldStyle = hasInitializedStyle() ? &style() : nullptr;
    174174    if (oldStyle && !oldStyle->lineClamp().isNone() && newStyle.lineClamp().isNone())
    175175        clearLineClamp();
    … …  
    215215
    216216    m_minPreferredLogicalWidth = m_maxPreferredLogicalWidth = 0;
    217     if (style()->width().isFixed() && style()->width().value() > 0)
    218         m_minPreferredLogicalWidth = m_maxPreferredLogicalWidth = adjustContentBoxLogicalWidthForBoxSizing(style()->width().value());
     217    if (style().width().isFixed() && style().width().value() > 0)
     218        m_minPreferredLogicalWidth = m_maxPreferredLogicalWidth = adjustContentBoxLogicalWidthForBoxSizing(style().width().value());
    219219    else
    220220        computeIntrinsicLogicalWidths(m_minPreferredLogicalWidth, m_maxPreferredLogicalWidth);
    221221
    222     if (style()->minWidth().isFixed() && style()->minWidth().value() > 0) {
    223         m_maxPreferredLogicalWidth = max(m_maxPreferredLogicalWidth, adjustContentBoxLogicalWidthForBoxSizing(style()->minWidth().value()));
    224         m_minPreferredLogicalWidth = max(m_minPreferredLogicalWidth, adjustContentBoxLogicalWidthForBoxSizing(style()->minWidth().value()));
    225     }
    226 
    227     if (style()->maxWidth().isFixed()) {
    228         m_maxPreferredLogicalWidth = min(m_maxPreferredLogicalWidth, adjustContentBoxLogicalWidthForBoxSizing(style()->maxWidth().value()));
    229         m_minPreferredLogicalWidth = min(m_minPreferredLogicalWidth, adjustContentBoxLogicalWidthForBoxSizing(style()->maxWidth().value()));
     222    if (style().minWidth().isFixed() && style().minWidth().value() > 0) {
     223        m_maxPreferredLogicalWidth = max(m_maxPreferredLogicalWidth, adjustContentBoxLogicalWidthForBoxSizing(style().minWidth().value()));
     224        m_minPreferredLogicalWidth = max(m_minPreferredLogicalWidth, adjustContentBoxLogicalWidthForBoxSizing(style().minWidth().value()));
     225    }
     226
     227    if (style().maxWidth().isFixed()) {
     228        m_maxPreferredLogicalWidth = min(m_maxPreferredLogicalWidth, adjustContentBoxLogicalWidthForBoxSizing(style().maxWidth().value()));
     229        m_minPreferredLogicalWidth = min(m_minPreferredLogicalWidth, adjustContentBoxLogicalWidthForBoxSizing(style().maxWidth().value()));
    230230    }
    231231
    … …  
    286286
    287287    LayoutRepainter repainter(*this, checkForRepaintDuringLayout());
    288     LayoutStateMaintainer statePusher(&view(), this, locationOffset(), hasTransform() || hasReflection() || style()->isFlippedBlocksWritingMode());
     288    LayoutStateMaintainer statePusher(&view(), this, locationOffset(), hasTransform() || hasReflection() || style().isFlippedBlocksWritingMode());
    289289
    290290    prepareShapesAndPaginationBeforeBlockLayout(relayoutChildren);
    … …  
    296296
    297297    if (previousSize != size()
    298         || (parent()->isDeprecatedFlexibleBox() && parent()->style()->boxOrient() == HORIZONTAL
    299         && parent()->style()->boxAlign() == BSTRETCH))
     298        || (parent()->isDeprecatedFlexibleBox() && parent()->style().boxOrient() == HORIZONTAL
     299        && parent()->style().boxAlign() == BSTRETCH))
    300300        relayoutChildren = true;
    301301
    … …  
    355355    for (RenderBox* child = iterator.first(); child; child = iterator.next()) {
    356356        // Check to see if this child flexes.
    357         if (!childDoesNotAffectWidthOrFlexing(child) && child->style()->boxFlex() > 0.0f) {
     357        if (!childDoesNotAffectWidthOrFlexing(child) && child->style().boxFlex() > 0.0f) {
    358358            // We always have to lay out flexible objects again, since the flex distribution
    359359            // may have changed, and we need to reallocate space.
    … …  
    362362                child->setChildNeedsLayout(MarkOnlyThis);
    363363            haveFlex = true;
    364             unsigned int flexGroup = child->style()->boxFlexGroup();
     364            unsigned flexGroup = child->style().boxFlexGroup();
    365365            if (lowestFlexGroup == 0)
    366366                lowestFlexGroup = flexGroup;
    … …  
    441441
    442442            // Update our height and overflow height.
    443             if (style()->boxAlign() == BBASELINE) {
     443            if (style().boxAlign() == BBASELINE) {
    444444                LayoutUnit ascent = child->firstLineBaseline();
    445445                if (ascent == -1)
    … …  
    463463
    464464        if (!iterator.first() && hasLineIfEmpty())
    465             setHeight(height() + lineHeight(true, style()->isHorizontalWritingMode() ? HorizontalLine : VerticalLine, PositionOfInteriorLineBoxes));
     465            setHeight(height() + lineHeight(true, style().isHorizontalWritingMode() ? HorizontalLine : VerticalLine, PositionOfInteriorLineBoxes));
    466466
    467467        setHeight(height() + toAdd);
    … …  
    476476        // Now that our height is actually known, we can place our boxes.
    477477        childIndex = 0;
    478         m_stretchingChildren = (style()->boxAlign() == BSTRETCH);
     478        m_stretchingChildren = (style().boxAlign() == BSTRETCH);
    479479        for (RenderBox* child = iterator.first(); child; child = iterator.next()) {
    480480            if (child->isOutOfFlowPositioned()) {
    … …  
    484484                if (childLayer->staticBlockPosition() != yPos) {
    485485                    childLayer->setStaticBlockPosition(yPos);
    486                     if (child->style()->hasStaticBlockPosition(style()->isHorizontalWritingMode()))
     486                    if (child->style().hasStaticBlockPosition(style().isHorizontalWritingMode()))
    487487                        child->setChildNeedsLayout(MarkOnlyThis);
    488488                }
    … …  
    492492            LayoutSize& childLayoutDelta = childLayoutDeltas[childIndex++];
    493493           
    494             if (child->style()->visibility() == COLLAPSE) {
     494            if (child->style().visibility() == COLLAPSE) {
    495495                // visibility: collapsed children do not participate in our positioning.
    496496                // But we need to lay them out.
    … …  
    515515            xPos += child->marginLeft();
    516516            LayoutUnit childY = yPos;
    517             switch (style()->boxAlign()) {
     517            switch (style().boxAlign()) {
    518518                case BCENTER:
    519519                    childY += child->marginTop() + max<LayoutUnit>(0, (contentHeight() - (child->height() + child->marginHeight())) / 2);
    … …  
    569569                    for (RenderBox* child = iterator.first(); child; child = iterator.next()) {
    570570                        if (allowedChildFlex(child, expanding, i))
    571                             totalFlex += child->style()->boxFlex();
     571                            totalFlex += child->style().boxFlex();
    572572                    }
    573573                    LayoutUnit spaceAvailableThisPass = groupRemainingSpace;
    … …  
    575575                        LayoutUnit allowedFlex = allowedChildFlex(child, expanding, i);
    576576                        if (allowedFlex) {
    577                             LayoutUnit projectedFlex = (allowedFlex == LayoutUnit::max()) ? allowedFlex : LayoutUnit(allowedFlex * (totalFlex / child->style()->boxFlex()));
     577                            LayoutUnit projectedFlex = (allowedFlex == LayoutUnit::max()) ? allowedFlex : LayoutUnit(allowedFlex * (totalFlex / child->style().boxFlex()));
    578578                            spaceAvailableThisPass = expanding ? min(spaceAvailableThisPass, projectedFlex) : max(spaceAvailableThisPass, projectedFlex);
    579579                        }
    … …  
    589589                    // Now distribute the space to objects.
    590590                    for (RenderBox* child = iterator.first(); child && spaceAvailableThisPass && totalFlex; child = iterator.next()) {
    591                         if (child->style()->visibility() == COLLAPSE)
     591                        if (child->style().visibility() == COLLAPSE)
    592592                            continue;
    593593
    594594                        if (allowedChildFlex(child, expanding, i)) {
    595                             LayoutUnit spaceAdd = LayoutUnit(spaceAvailableThisPass * (child->style()->boxFlex() / totalFlex));
     595                            LayoutUnit spaceAdd = LayoutUnit(spaceAvailableThisPass * (child->style().boxFlex() / totalFlex));
    596596                            if (spaceAdd) {
    597597                                child->setOverrideLogicalContentWidth(contentWidthForChild(child) + spaceAdd);
    … …  
    604604                            groupRemainingSpace -= spaceAdd;
    605605
    606                             totalFlex -= child->style()->boxFlex();
     606                            totalFlex -= child->style().boxFlex();
    607607                        }
    608608                    }
    … …  
    631631    RenderBlock::finishDelayUpdateScrollInfo();
    632632
    633     if (remainingSpace > 0 && ((style()->isLeftToRightDirection() && style()->boxPack() != Start)
    634         || (!style()->isLeftToRightDirection() && style()->boxPack() != End))) {
     633    if (remainingSpace > 0 && ((style().isLeftToRightDirection() && style().boxPack() != Start)
     634        || (!style().isLeftToRightDirection() && style().boxPack() != End))) {
    635635        // Children must be repositioned.
    636636        LayoutUnit offset = 0;
    637         if (style()->boxPack() == Justify) {
     637        if (style().boxPack() == Justify) {
    638638            // Determine the total number of children.
    639639            int totalChildren = 0;
    … …  
    666666            }
    667667        } else {
    668             if (style()->boxPack() == Center)
     668            if (style().boxPack() == Center)
    669669                offset += remainingSpace / 2;
    670670            else // END for LTR, START for RTL
    … …  
    702702    // We confine the line clamp ugliness to vertical flexible boxes (thus keeping it out of
    703703    // mainstream block layout); this is not really part of the XUL box model.
    704     bool haveLineClamp = !style()->lineClamp().isNone();
     704    bool haveLineClamp = !style().lineClamp().isNone();
    705705    if (haveLineClamp)
    706706        applyLineClamp(iterator, relayoutChildren);
    … …  
    731731                if (childLayer->staticBlockPosition() != height()) {
    732732                    childLayer->setStaticBlockPosition(height());
    733                     if (child->style()->hasStaticBlockPosition(style()->isHorizontalWritingMode()))
     733                    if (child->style().hasStaticBlockPosition(style().isHorizontalWritingMode()))
    734734                        child->setChildNeedsLayout(MarkOnlyThis);
    735735                }
    … …  
    739739            LayoutSize& childLayoutDelta = childLayoutDeltas[childIndex++];
    740740
    741             if (child->style()->visibility() == COLLAPSE) {
     741            if (child->style().visibility() == COLLAPSE) {
    742742                // visibility: collapsed children do not participate in our positioning.
    743743                // But we need to lay them down.
    … …  
    760760            // We can place the child now, using our value of box-align.
    761761            LayoutUnit childX = borderLeft() + paddingLeft();
    762             switch (style()->boxAlign()) {
     762            switch (style().boxAlign()) {
    763763                case BCENTER:
    764764                case BBASELINE: // Baseline just maps to center for vertical boxes
    … …  
    766766                    break;
    767767                case BEND:
    768                     if (!style()->isLeftToRightDirection())
     768                    if (!style().isLeftToRightDirection())
    769769                        childX += child->marginLeft();
    770770                    else
    … …  
    772772                    break;
    773773                default: // BSTART/BSTRETCH
    774                     if (style()->isLeftToRightDirection())
     774                    if (style().isLeftToRightDirection())
    775775                        childX += child->marginLeft();
    776776                    else
    … …  
    788788
    789789        if (!iterator.first() && hasLineIfEmpty())
    790             setHeight(height() + lineHeight(true, style()->isHorizontalWritingMode() ? HorizontalLine : VerticalLine, PositionOfInteriorLineBoxes));
     790            setHeight(height() + lineHeight(true, style().isHorizontalWritingMode() ? HorizontalLine : VerticalLine, PositionOfInteriorLineBoxes));
    791791
    792792        setHeight(height() + toAdd);
    … …  
    830830                    for (RenderBox* child = iterator.first(); child; child = iterator.next()) {
    831831                        if (allowedChildFlex(child, expanding, i))
    832                             totalFlex += child->style()->boxFlex();
     832                            totalFlex += child->style().boxFlex();
    833833                    }
    834834                    LayoutUnit spaceAvailableThisPass = groupRemainingSpace;
    … …  
    836836                        LayoutUnit allowedFlex = allowedChildFlex(child, expanding, i);
    837837                        if (allowedFlex) {
    838                             LayoutUnit projectedFlex = (allowedFlex == LayoutUnit::max()) ? allowedFlex : static_cast<LayoutUnit>(allowedFlex * (totalFlex / child->style()->boxFlex()));
     838                            LayoutUnit projectedFlex = (allowedFlex == LayoutUnit::max()) ? allowedFlex : static_cast<LayoutUnit>(allowedFlex * (totalFlex / child->style().boxFlex()));
    839839                            spaceAvailableThisPass = expanding ? min(spaceAvailableThisPass, projectedFlex) : max(spaceAvailableThisPass, projectedFlex);
    840840                        }
    … …  
    851851                    for (RenderBox* child = iterator.first(); child && spaceAvailableThisPass && totalFlex; child = iterator.next()) {
    852852                        if (allowedChildFlex(child, expanding, i)) {
    853                             LayoutUnit spaceAdd = static_cast<LayoutUnit>(spaceAvailableThisPass * (child->style()->boxFlex() / totalFlex));
     853                            LayoutUnit spaceAdd = static_cast<LayoutUnit>(spaceAvailableThisPass * (child->style().boxFlex() / totalFlex));
    854854                            if (spaceAdd) {
    855855                                child->setOverrideLogicalContentHeight(contentHeightForChild(child) + spaceAdd);
    … …  
    862862                            groupRemainingSpace -= spaceAdd;
    863863
    864                             totalFlex -= child->style()->boxFlex();
     864                            totalFlex -= child->style().boxFlex();
    865865                        }
    866866                    }
    … …  
    889889    RenderBlock::finishDelayUpdateScrollInfo();
    890890
    891     if (style()->boxPack() != Start && remainingSpace > 0) {
     891    if (style().boxPack() != Start && remainingSpace > 0) {
    892892        // Children must be repositioned.
    893893        LayoutUnit offset = 0;
    894         if (style()->boxPack() == Justify) {
     894        if (style().boxPack() == Justify) {
    895895            // Determine the total number of children.
    896896            int totalChildren = 0;
    … …  
    923923            }
    924924        } else {
    925             if (style()->boxPack() == Center)
     925            if (style().boxPack() == Center)
    926926                offset += remainingSpace / 2;
    927927            else // END
    … …  
    949949
    950950        child->clearOverrideSize();
    951         if (relayoutChildren || (child->isReplaced() && (child->style()->width().isPercent() || child->style()->height().isPercent()))
    952             || (child->style()->height().isAuto() && child->isRenderBlockFlow())) {
     951        if (relayoutChildren || (child->isReplaced() && (child->style().width().isPercent() || child->style().height().isPercent()))
     952            || (child->style().height().isAuto() && child->isRenderBlockFlow())) {
    953953            child->setChildNeedsLayout(MarkOnlyThis);
    954954
    … …  
    960960        }
    961961        child->layoutIfNeeded();
    962         if (child->style()->height().isAuto() && child->isRenderBlockFlow())
     962        if (child->style().height().isAuto() && child->isRenderBlockFlow())
    963963            maxLineCount = max(maxLineCount, toRenderBlockFlow(child)->lineCount());
    964964    }
    … …  
    966966    // Get the number of lines and then alter all block flow children with auto height to use the
    967967    // specified height. We always try to leave room for at least one line.
    968     LineClampValue lineClamp = style()->lineClamp();
     968    LineClampValue lineClamp = style().lineClamp();
    969969    int numVisibleLines = lineClamp.isPercentage() ? max(1, (maxLineCount + 1) * lineClamp.value() / 100) : lineClamp.value();
    970970    if (numVisibleLines >= maxLineCount)
    … …  
    972972
    973973    for (RenderBox* child = iterator.first(); child; child = iterator.next()) {
    974         if (childDoesNotAffectWidthOrFlexing(child) || !child->style()->height().isAuto() || !child->isRenderBlockFlow())
     974        if (childDoesNotAffectWidthOrFlexing(child) || !child->style().height().isAuto() || !child->isRenderBlockFlow())
    975975            continue;
    976976
    … …  
    989989
    990990        // FIXME: For now don't support RTL.
    991         if (style()->direction() != LTR)
     991        if (style().direction() != LTR)
    992992            continue;
    993993
    … …  
    10041004        DEFINE_STATIC_LOCAL(AtomicString, ellipsisAndSpaceStr, (ellipsisAndSpace, 2));
    10051005        DEFINE_STATIC_LOCAL(AtomicString, ellipsisStr, (&horizontalEllipsis, 1));
    1006         const RenderStyle& lineStyle = numVisibleLines == 1 ? *firstLineStyle() : *style();
     1006        const RenderStyle& lineStyle = numVisibleLines == 1 ? firstLineStyle() : style();
    10071007        const Font& font = lineStyle.font();
    10081008
    … …  
    10101010        LayoutUnit totalWidth;
    10111011        InlineBox* anchorBox = lastLine->lastChild();
    1012         if (anchorBox && anchorBox->renderer().style()->isLink())
    1013             totalWidth = anchorBox->logicalWidth() + font.width(constructTextRun(this, font, ellipsisAndSpace, 2, *style()));
     1012        if (anchorBox && anchorBox->renderer().style().isLink())
     1013            totalWidth = anchorBox->logicalWidth() + font.width(constructTextRun(this, font, ellipsisAndSpace, 2, style()));
    10141014        else {
    10151015            anchorBox = 0;
    1016             totalWidth = font.width(constructTextRun(this, font, &horizontalEllipsis, 1, *style()));
     1016            totalWidth = font.width(constructTextRun(this, font, &horizontalEllipsis, 1, style()));
    10171017        }
    10181018
    … …  
    10221022
    10231023        // FIXME: Directions of src/destBlock could be different from our direction and from one another.
    1024         if (!srcBlock.style()->isLeftToRightDirection())
     1024        if (!srcBlock.style().isLeftToRightDirection())
    10251025            continue;
    10261026
    1027         bool leftToRight = destBlock.style()->isLeftToRightDirection();
     1027        bool leftToRight = destBlock.style().isLeftToRightDirection();
    10281028        if (!leftToRight)
    10291029            continue;
    … …  
    10491049
    10501050        child->clearOverrideSize();
    1051         if ((child->isReplaced() && (child->style()->width().isPercent() || child->style()->height().isPercent()))
    1052             || (child->style()->height().isAuto() && child->isRenderBlock())) {
     1051        if ((child->isReplaced() && (child->style().width().isPercent() || child->style().height().isPercent()))
     1052            || (child->style().height().isAuto() && child->isRenderBlock())) {
    10531053            child->setChildNeedsLayout();
    10541054
    … …  
    10711071LayoutUnit RenderDeprecatedFlexibleBox::allowedChildFlex(RenderBox* child, bool expanding, unsigned int group)
    10721072{
    1073     if (childDoesNotAffectWidthOrFlexing(child) || child->style()->boxFlex() == 0.0f || child->style()->boxFlexGroup() != group)
     1073    if (childDoesNotAffectWidthOrFlexing(child) || child->style().boxFlex() == 0.0f || child->style().boxFlexGroup() != group)
    10741074        return 0;
    10751075
    … …  
    10791079            LayoutUnit maxWidth = LayoutUnit::max();
    10801080            LayoutUnit width = contentWidthForChild(child);
    1081             if (!child->style()->maxWidth().isUndefined() && child->style()->maxWidth().isFixed())
    1082                 maxWidth = child->style()->maxWidth().value();
    1083             else if (child->style()->maxWidth().type() == Intrinsic)
     1081            if (!child->style().maxWidth().isUndefined() && child->style().maxWidth().isFixed())
     1082                maxWidth = child->style().maxWidth().value();
     1083            else if (child->style().maxWidth().type() == Intrinsic)
    10841084                maxWidth = child->maxPreferredLogicalWidth();
    1085             else if (child->style()->maxWidth().type() == MinIntrinsic)
     1085            else if (child->style().maxWidth().type() == MinIntrinsic)
    10861086                maxWidth = child->minPreferredLogicalWidth();
    10871087            if (maxWidth == LayoutUnit::max())
    … …  
    10921092            LayoutUnit maxHeight = LayoutUnit::max();
    10931093            LayoutUnit height = contentHeightForChild(child);
    1094             if (!child->style()->maxHeight().isUndefined() && child->style()->maxHeight().isFixed())
    1095                 maxHeight = child->style()->maxHeight().value();
     1094            if (!child->style().maxHeight().isUndefined() && child->style().maxHeight().isFixed())
     1095                maxHeight = child->style().maxHeight().value();
    10961096            if (maxHeight == LayoutUnit::max())
    10971097                return maxHeight;
    … …  
    11041104        LayoutUnit minWidth = child->minPreferredLogicalWidth();
    11051105        LayoutUnit width = contentWidthForChild(child);
    1106         if (child->style()->minWidth().isFixed())
    1107             minWidth = child->style()->minWidth().value();
    1108         else if (child->style()->minWidth().type() == Intrinsic)
     1106        if (child->style().minWidth().isFixed())
     1107            minWidth = child->style().minWidth().value();
     1108        else if (child->style().minWidth().type() == Intrinsic)
    11091109            minWidth = child->maxPreferredLogicalWidth();
    1110         else if (child->style()->minWidth().type() == MinIntrinsic)
     1110        else if (child->style().minWidth().type() == MinIntrinsic)
    11111111            minWidth = child->minPreferredLogicalWidth();
    1112         else if (child->style()->minWidth().type() == Auto)
     1112        else if (child->style().minWidth().type() == Auto)
    11131113            minWidth = 0;
    11141114
    … …  
    11161116        return allowedShrinkage;
    11171117    } else {
    1118         Length minHeight = child->style()->minHeight();
     1118        Length minHeight = child->style().minHeight();
    11191119        if (minHeight.isFixed() || minHeight.isAuto()) {
    1120             LayoutUnit minHeight = child->style()->minHeight().value();
     1120            LayoutUnit minHeight = child->style().minHeight().value();
    11211121            LayoutUnit height = contentHeightForChild(child);
    11221122            LayoutUnit allowedShrinkage = min<LayoutUnit>(0, minHeight - height);
Note: See TracChangeset for help on using the changeset viewer.