Use references in RenderBlock and RenderBlockFlow methods

Source/core/rendering/RenderFieldset.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006 Apple Computer, Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
@@ skipping 56 matching lines @@
     RenderBox* legend = findLegend();
     if (legend) {
         if (relayoutChildren)
             legend->setNeedsLayoutAndFullPaintInvalidation();
         legend->layoutIfNeeded();
 
         LayoutUnit logicalLeft;
         if (style()->isLeftToRightDirection()) {
             switch (legend->style()->textAlign()) {
             case CENTER:
-                logicalLeft = (logicalWidth() - logicalWidthForChild(legend)) / 2;
+                logicalLeft = (logicalWidth() - logicalWidthForChild(*legend)) / 2;
                 break;
             case RIGHT:
-                logicalLeft = logicalWidth() - borderEnd() - paddingEnd() - logicalWidthForChild(legend);
+                logicalLeft = logicalWidth() - borderEnd() - paddingEnd() - logicalWidthForChild(*legend);
                 break;
             default:
-                logicalLeft = borderStart() + paddingStart() + marginStartForChild(legend);
+                logicalLeft = borderStart() + paddingStart() + marginStartForChild(*legend);
                 break;
             }
         } else {
             switch (legend->style()->textAlign()) {
             case LEFT:
                 logicalLeft = borderStart() + paddingStart();
                 break;
             case CENTER: {
                 // Make sure that the extra pixel goes to the end side in RTL (since it went to the end side
                 // in LTR).
-                LayoutUnit centeredWidth = logicalWidth() - logicalWidthForChild(legend);
+                LayoutUnit centeredWidth = logicalWidth() - logicalWidthForChild(*legend);
                 logicalLeft = centeredWidth - centeredWidth / 2;
                 break;
             }
             default:
-                logicalLeft = logicalWidth() - borderStart() - paddingStart() - marginStartForChild(legend) - logicalWidthForChild(legend);
+                logicalLeft = logicalWidth() - borderStart() - paddingStart() - marginStartForChild(*legend) - logicalWidthForChild(*legend);
                 break;
             }
         }
 
-        setLogicalLeftForChild(legend, logicalLeft);
+        setLogicalLeftForChild(*legend, logicalLeft);
 
         LayoutUnit fieldsetBorderBefore = borderBefore();
-        LayoutUnit legendLogicalHeight = logicalHeightForChild(legend);
+        LayoutUnit legendLogicalHeight = logicalHeightForChild(*legend);
 
         LayoutUnit legendLogicalTop;
         LayoutUnit collapsedLegendExtent;
         // FIXME: We need to account for the legend's margin before too.
         if (fieldsetBorderBefore > legendLogicalHeight) {
             // The <legend> is smaller than the associated fieldset before border
             // so the latter determines positioning of the <legend>. The sizing depends
             // on the legend's margins as we want to still follow the author's cues.
             // Firefox completely ignores the margins in this case which seems wrong.
             legendLogicalTop = (fieldsetBorderBefore - legendLogicalHeight) / 2;
-            collapsedLegendExtent = max<LayoutUnit>(fieldsetBorderBefore, legendLogicalTop + legendLogicalHeight + marginAfterForChild(legend));
-        } else
-            collapsedLegendExtent = legendLogicalHeight + marginAfterForChild(legend);
+            collapsedLegendExtent = max<LayoutUnit>(fieldsetBorderBefore, legendLogicalTop + legendLogicalHeight + marginAfterForChild(*legend));
+        } else {
+            collapsedLegendExtent = legendLogicalHeight + marginAfterForChild(*legend);
+        }
 
-        setLogicalTopForChild(legend, legendLogicalTop);
+        setLogicalTopForChild(*legend, legendLogicalTop);
         setLogicalHeight(paddingBefore() + collapsedLegendExtent);
     }
     return legend;
 }
 
 RenderBox* RenderFieldset::findLegend(FindLegendOption option) const
 {
     for (RenderObject* legend = firstChild(); legend; legend = legend->nextSibling()) {
         if (option == IgnoreFloatingOrOutOfFlow && legend->isFloatingOrOutOfFlowPositioned())
             continue;
 
         if (isHTMLLegendElement(legend->node()))
             return toRenderBox(legend);
     }
     return 0;
 }
 
 void RenderFieldset::paintBoxDecorationBackground(const PaintInfo& paintInfo, const LayoutPoint& paintOffset)
 {
     FieldsetPainter(*this).paintBoxDecorationBackground(paintInfo, paintOffset);
 }
 
 void RenderFieldset::paintMask(const PaintInfo& paintInfo, const LayoutPoint& paintOffset)
 {
     FieldsetPainter(*this).paintMask(paintInfo, paintOffset);
 }
 
 } // namespace blink