Move hasRenderedNonAnonymousDescendantsWithHeight() to VisibleUnits.cpp from Position.cpp

Source/core/editing/Position.cpp

Index: Source/core/editing/Position.cpp
diff --git a/Source/core/editing/Position.cpp b/Source/core/editing/Position.cpp
index 332cf9d84587b6c3943dfcce73c8fe03a109a7d7..2bf3ceef08fdedb80042536c4c41131a1b455734 100644
--- a/Source/core/editing/Position.cpp
+++ b/Source/core/editing/Position.cpp
@@ -42,7 +42,6 @@
 #include "core/layout/LayoutBlock.h"
 #include "core/layout/LayoutInline.h"
 #include "core/layout/LayoutText.h"
-#include "core/layout/line/InlineIterator.h"
 #include "core/layout/line/InlineTextBox.h"
 #include "wtf/text/CString.h"
 #include <stdio.h>
@@ -528,27 +527,6 @@ PositionAlgorithm<Strategy> PositionAlgorithm<Strategy>::downstream(EditingBound
     return mostBackwardCaretPosition(*this, rule);
 }
 
-static int boundingBoxLogicalHeight(LayoutObject *o, const IntRect &rect)
-{
-    return o->style()->isHorizontalWritingMode() ? rect.height() : rect.width();
-}
-
-// TODO(yosin) We should move |hasRenderedNonAnonymousDescendantsWithHeight|
-// to "VisibleUnits.cpp" to reduce |LayoutObject| dependency in "Position.cpp"
-bool hasRenderedNonAnonymousDescendantsWithHeight(LayoutObject* layoutObject)
-{
-    LayoutObject* stop = layoutObject->nextInPreOrderAfterChildren();
-    for (LayoutObject *o = layoutObject->slowFirstChild(); o && o != stop; o = o->nextInPreOrder()) {
-        if (o->nonPseudoNode()) {
-            if ((o->isText() && boundingBoxLogicalHeight(o, toLayoutText(o)->linesBoundingBox()))
-                || (o->isBox() && toLayoutBox(o)->pixelSnappedLogicalHeight())
-                || (o->isLayoutInline() && isEmptyInline(LineLayoutItem(o)) && boundingBoxLogicalHeight(o, toLayoutInline(o)->linesBoundingBox())))
-                return true;
-        }
-    }
-    return false;
-}
-
 template <typename Strategy>
 Node* PositionAlgorithm<Strategy>::rootUserSelectAllForNode(Node* node)
 {