Get rid of forward declaration of ComputeInlineBoxPositionForTextNode()

2941213002: Get rid of forward declaration of ComputeInlineBoxPositionForTextNode()

This patch moves implementation of |ComputeInlineBoxPositionForTextNode()| at
forward declaration to get rid of forward declaration for improving code health.

This is follow-up the patch[1] which inttroduces it.


[1] http://crrev.com/2941013003 Introduce ComputeInlineBoxPositionForTextNode()

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

Review-Url: https://codereview.chromium.org/2941213002
Cr-Commit-Position: refs/heads/master@{#479986}
Committed: https://chromium.googlesource.com/chromium/src/+/f27177df9fa6d949bd33fd6095f8c8a1481b1890

[yosin_UTC9, 2017-06-16 05:37:15]

The CQ bit was checked by yosin@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-16 05:37:27]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yosin_UTC9, 2017-06-16 06:43:09]

Description was changed from

==========
2017-06-16T14:34:05

BUG=707656

2017-06-16T14:34:01
==========

to

==========
2941213002: Get rid of forward declaration of
ComputeInlineBoxPositionForTextNode()

This patch moves implementation of |ComputeInlineBoxPositionForTextNode()| at
forward declaration to get rid of forward declaration for improving code health.

This is follow-up the patch[1] which inttroduces it.


[1] http://crrev.com/2941013003 Introduce ComputeInlineBoxPositionForTextNode()

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

[yosin_UTC9, 2017-06-16 06:43:22]

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

[yosin_UTC9, 2017-06-16 06:43:22]

PTAL

[commit-bot: I haz the power, 2017-06-16 07:24:07]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-16 07:24:08]

Dry run: This issue passed the CQ dry run.

[yoichio, 2017-06-16 07:49:10]

lgtm

[yosin_UTC9, 2017-06-16 07:53:33]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2017-06-16 07:53:49]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-16 07:56:47]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1497599613325190, "parent_rev":
"89174ae3149c9b72dea9f97dc49e9d223fb6936d", "commit_rev":
"f27177df9fa6d949bd33fd6095f8c8a1481b1890"}

[commit-bot: I haz the power, 2017-06-16 07:57:02]

Description was changed from

==========
2941213002: Get rid of forward declaration of
ComputeInlineBoxPositionForTextNode()

This patch moves implementation of |ComputeInlineBoxPositionForTextNode()| at
forward declaration to get rid of forward declaration for improving code health.

This is follow-up the patch[1] which inttroduces it.


[1] http://crrev.com/2941013003 Introduce ComputeInlineBoxPositionForTextNode()

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

to

==========
2941213002: Get rid of forward declaration of
ComputeInlineBoxPositionForTextNode()

This patch moves implementation of |ComputeInlineBoxPositionForTextNode()| at
forward declaration to get rid of forward declaration for improving code health.

This is follow-up the patch[1] which inttroduces it.


[1] http://crrev.com/2941013003 Introduce ComputeInlineBoxPositionForTextNode()

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

Review-Url: https://codereview.chromium.org/2941213002
Cr-Commit-Position: refs/heads/master@{#479986}
Committed:
https://chromium.googlesource.com/chromium/src/+/f27177df9fa6d949bd33fd6095f8...
==========

[commit-bot: I haz the power, 2017-06-16 07:57:04]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/f27177df9fa6d949bd33fd6095f8...