Get rid of FrameSelection::moveTo() taking two VisiblePosition objects

Get rid of FrameSelection::moveTo() taking two VisiblePosition objects

This patch gets rid of |FrameSelection::moveTo()| taking two |VisiblePosition|
objects since its call site is only one for improving code health.

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

Committed: https://crrev.com/e8087c7e76b36995f2a98c5ac0c8552ea7962536
Cr-Commit-Position: refs/heads/master@{#414266}

[yosin_UTC9, 2016-08-24 08:12:57]

Description was changed from

==========
2016-08-24T17:07:58

BUG=
==========

to

==========
Get rid of unused member function FrameSelection::moveTo() taking two
VisiblePosition objects

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

[yosin_UTC9, 2016-08-24 08:13:02]

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

[commit-bot: I haz the power, 2016-08-24 08:13:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yosin_UTC9, 2016-08-24 08:14:12]

Description was changed from

==========
Get rid of unused member function FrameSelection::moveTo() taking two
VisiblePosition objects

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

to

==========
Get rid of FrameSelection::moveTo() taking two VisiblePosition objects

This patch gets rid of |FrameSelection::moveTo()| taking two |VisiblePosition|
objects since its call site is only one for improving code health.

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

[yosin_UTC9, 2016-08-24 08:45:39]

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

[yosin_UTC9, 2016-08-24 08:45:40]

PTAL

[Xiaocheng, 2016-08-24 09:04:05]

lgtm

[commit-bot: I haz the power, 2016-08-24 10:03:26]

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

[commit-bot: I haz the power, 2016-08-24 10:03:28]

Dry run: This issue passed the CQ dry run.

[yosin_UTC9, 2016-08-25 01:26:36]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-08-25 01:27:33]

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, 2016-08-25 02:25:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-25 02:28:14]

Description was changed from

==========
Get rid of FrameSelection::moveTo() taking two VisiblePosition objects

This patch gets rid of |FrameSelection::moveTo()| taking two |VisiblePosition|
objects since its call site is only one for improving code health.

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

to

==========
Get rid of FrameSelection::moveTo() taking two VisiblePosition objects

This patch gets rid of |FrameSelection::moveTo()| taking two |VisiblePosition|
objects since its call site is only one for improving code health.

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

Committed: https://crrev.com/e8087c7e76b36995f2a98c5ac0c8552ea7962536
Cr-Commit-Position: refs/heads/master@{#414266}
==========

[commit-bot: I haz the power, 2016-08-25 02:28:15]

Patchset 1 (id:??) landed as
https://crrev.com/e8087c7e76b36995f2a98c5ac0c8552ea7962536
Cr-Commit-Position: refs/heads/master@{#414266}