Get rid of createVisibleSelection() taking one Position

Get rid of flat tree version of createVisibleSelection() taking one ePosition

This patch gets rid of flat tree version of |createVisibleSelection()| taking
one |Position| by replacing with |SelectionInDOMTree| version to reduce number
of overloads for improving code health.

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

Committed: https://crrev.com/7fffd89947118c4719ccfb7afceae7ca66d93f5b
Cr-Commit-Position: refs/heads/master@{#427280}

[yosin_UTC9, 2016-10-21 07:55:42]

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

[commit-bot: I haz the power, 2016-10-21 07:55:59]

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-10-21 08:45:28]

Description was changed from

==========
2016-10-21T16:50:37

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

to

==========
Get rid of flat tree version of createVisibleSelection() taking one ePosition

This patch gets rid of flat tree version of |createVisibleSelection()| taking
one |Position| by replacing with |SelectionInDOMTree| version to reduce number 
of overloads for improving code health.

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

[commit-bot: I haz the power, 2016-10-21 09:06:33]

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

[commit-bot: I haz the power, 2016-10-21 09:06:34]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[yosin_UTC9, 2016-10-24 08:03:17]

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

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

Dry run: 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-10-24 08:26:15]

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

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

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[yosin_UTC9, 2016-10-24 08:44:27]

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

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

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-10-24 09:34:35]

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

[yosin_UTC9, 2016-10-24 09:34:36]

PTAL

Similar to http://crrev.com/2437873008, which getting rig of
createVisibleSelection() taking two VisiblePosition.

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

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

[commit-bot: I haz the power, 2016-10-24 10:50:09]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[tkent, 2016-10-25 03:37:48]

lgtm

[yoichio, 2016-10-25 03:45:33]

lgtm

[yosin_UTC9, 2016-10-25 04:13:58]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-10-25 04:14:15]

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-10-25 05:21:22]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-25 05:23:23]

Description was changed from

==========
Get rid of flat tree version of createVisibleSelection() taking one ePosition

This patch gets rid of flat tree version of |createVisibleSelection()| taking
one |Position| by replacing with |SelectionInDOMTree| version to reduce number 
of overloads for improving code health.

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

to

==========
Get rid of flat tree version of createVisibleSelection() taking one ePosition

This patch gets rid of flat tree version of |createVisibleSelection()| taking
one |Position| by replacing with |SelectionInDOMTree| version to reduce number
of overloads for improving code health.

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

Committed: https://crrev.com/7fffd89947118c4719ccfb7afceae7ca66d93f5b
Cr-Commit-Position: refs/heads/master@{#427280}
==========

[commit-bot: I haz the power, 2016-10-25 05:23:24]

Patchset 3 (id:??) landed as
https://crrev.com/7fffd89947118c4719ccfb7afceae7ca66d93f5b
Cr-Commit-Position: refs/heads/master@{#427280}