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

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

This patch moves |hasRenderedNonAnonymousDescendantsWithHeight()| form
"Position.cpp" to "VisibleUnits.cpp" to reduce |InlineBox| dependency in
"Position.cpp" for improving code health, since |hasRender...()| is independent
from an implementation of |PositionAlgorithm| template class.

BUG=518738
TEST=n/a; no behavior changes

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200896

[yosin_UTC9, 2015-08-20 10:03:35]

yosin@chromium.org changed reviewers:
+ hajimehoshi@chromium.org, yoichio@chromium.org

[hajimehoshi, 2015-08-20 11:11:18]

lgtm

[yosin_UTC9, 2015-08-20 11:22:01]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2015-08-20 11:22:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1288873009/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1288873009/1

[commit-bot: I haz the power, 2015-08-20 12:06:24]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200896