Add version of TextIterator constructor that takes EphemeralRange

Add version of TextIterator constructor that takes EphemeralRange

Currently TextIterator's constructor takes two Positions (start and end). It
came up during review of https://codereview.chromium.org/2901213008 that it
would be helpful to have another version of the constructor that instead takes
an EphemeralRange.

BUG=

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

[rlanday, 2017-05-30 22:16:31]

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

[rlanday, 2017-05-30 22:16:35]

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

[commit-bot: I haz the power, 2017-05-30 22:17:33]

Dry run: CQ is trying da patch.

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

[Xiaocheng, 2017-05-30 22:30:31]

lgtm

[commit-bot: I haz the power, 2017-05-31 01:01:40]

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

[commit-bot: I haz the power, 2017-05-31 01:01:41]

Dry run: This issue passed the CQ dry run.

[yosin_UTC9, 2017-05-31 01:52:50]

lgtm

Thanks for doing this!

Once the patch[1] is committed, we can deprecate |Position| version of ctor.

[1] http://crrev.com/2912053002: Call TextIterator constructor with proper range

[yosin_UTC9, 2017-05-31 01:52:53]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2017-05-31 01:53:44]

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-05-31 01:59:11]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496195573258120, "parent_rev":
"d564808ec725e2eb36bef109d17fe641885e0ad0", "commit_rev":
"e50a33061a1efd977881cc37eb53e762497393c9"}

[commit-bot: I haz the power, 2017-05-31 01:59:20]

Description was changed from

==========
Add version of TextIterator constructor that takes EphemeralRange

Currently TextIterator's constructor takes two Positions (start and end). It
came up during review of https://codereview.chromium.org/2901213008 that it
would be helpful to have another version of the constructor that instead takes
an EphemeralRange.

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

to

==========
Add version of TextIterator constructor that takes EphemeralRange

Currently TextIterator's constructor takes two Positions (start and end). It
came up during review of https://codereview.chromium.org/2901213008 that it
would be helpful to have another version of the constructor that instead takes
an EphemeralRange.

BUG=

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

[commit-bot: I haz the power, 2017-05-31 01:59:21]

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