Have ElementTraversal / NodeTraversal's next() methods take a reference

Source/core/rendering/RenderNamedFlowThread.cpp

 /*
  * Copyright (C) 2012 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 550 matching lines @@
 static bool boxIntersectsRegion(LayoutUnit logicalTopForBox, LayoutUnit logicalBottomForBox, LayoutUnit logicalTopForRegion, LayoutUnit logicalBottomForRegion)
 {
     bool regionIsEmpty = logicalBottomForRegion != LayoutUnit::max() && logicalTopForRegion != LayoutUnit::min()
         && (logicalBottomForRegion - logicalTopForRegion) <= 0;
     return  (logicalBottomForBox - logicalTopForBox) > 0
         && !regionIsEmpty
         && logicalTopForBox < logicalBottomForRegion && logicalTopForRegion < logicalBottomForBox;
 }
 
 // Retrieve the next node to be visited while computing the ranges inside a region.
-static Node* nextNodeInsideContentNode(const Node* currNode, const Node* contentNode)
+static Node* nextNodeInsideContentNode(const Node& currNode, const Node* contentNode)
 {
-    ASSERT(currNode);
     ASSERT(contentNode && contentNode->inNamedFlow());
 
-    if (currNode->renderer() && currNode->renderer()->isSVGRoot())
-        return NodeTraversal::nextSkippingChildren(currNode, contentNode);
+    if (currNode.renderer() && currNode.renderer()->isSVGRoot())
+        return NodeTraversal::nextSkippingChildren(&currNode, contentNode);
     return NodeTraversal::next(currNode, contentNode);
 }
 
 void RenderNamedFlowThread::getRanges(Vector<RefPtr<Range> >& rangeObjects, const RenderRegion* region) const
 {
     LayoutUnit logicalTopForRegion;
     LayoutUnit logicalBottomForRegion;
 
     // extend the first region top to contain everything up to its logical height
     if (region->isFirstRegion())
@@ skipping 20 matching lines @@
         if (!contentNode->renderer())
             continue;
 
         RefPtr<Range> range = Range::create(contentNode->document());
         bool foundStartPosition = false;
         bool startsAboveRegion = true;
         bool endsBelowRegion = true;
         bool skipOverOutsideNodes = false;
         Node* lastEndNode = 0;
 
-        for (Node* node = contentNode; node; node = nextNodeInsideContentNode(node, contentNode)) {
+        for (Node* node = contentNode; node; node = nextNodeInsideContentNode(*node, contentNode)) {
             RenderObject* renderer = node->renderer();
             if (!renderer)
                 continue;
 
             LayoutRect boundingBox;
             if (renderer->isRenderInline()) {
                 boundingBox = toRenderInline(renderer)->linesBoundingBox();
             } else if (renderer->isText()) {
                 boundingBox = toRenderText(renderer)->linesBoundingBox();
             } else {
@@ skipping 93 matching lines @@
                     lastEndNode = node;
                 }
             }
         }
         if (foundStartPosition || skipOverOutsideNodes)
             rangeObjects.append(range);
     }
 }
 
 }