Expand TextIterator::Initialize() into constructor

Expand TextIterator::Initialize() into constructor

This patch expands |TextIterator::Initialize()| into constructor where the
only call sites to simplify code for improving code health.

This patch is a follow-up of patch[1], which makes call sites of
|Initialize()| to just one.

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

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

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

[yosin_UTC9, 2017-05-31 07:17:11]

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

[commit-bot: I haz the power, 2017-05-31 07:17:33]

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-05-31 07:27:08]

Description was changed from

==========
2017-05-31T16:15:33

BUG=

2017-05-31T16:15:27
==========

to

==========
Expand TextIterator::Initialize() into constructor

This patch expands |TextIterator::Initialize()| into constructor where the
only call sites to simplify code for improving code health.

This patch is a follow-up of patch[1], which makes call sites of
|Initialize()| to just one.

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

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

[yosin_UTC9, 2017-05-31 07:48:34]

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

[yosin_UTC9, 2017-05-31 07:48:34]

PTAL

[yoichio, 2017-05-31 07:51:38]

lgtm

[yosin_UTC9, 2017-05-31 08:02:06]

The CQ bit was unchecked by yosin@chromium.org

[yosin_UTC9, 2017-05-31 08:02:08]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2017-05-31 08:02:13]

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 08:56:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496217728310240, "parent_rev":
"944eab08ac07c46cb5ce2406362f7e376e4c4dec", "commit_rev":
"e15b8a7a2ed4a844796968f23113955aa4403a7a"}

[commit-bot: I haz the power, 2017-05-31 08:56:16]

Description was changed from

==========
Expand TextIterator::Initialize() into constructor

This patch expands |TextIterator::Initialize()| into constructor where the
only call sites to simplify code for improving code health.

This patch is a follow-up of patch[1], which makes call sites of
|Initialize()| to just one.

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

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

to

==========
Expand TextIterator::Initialize() into constructor

This patch expands |TextIterator::Initialize()| into constructor where the
only call sites to simplify code for improving code health.

This patch is a follow-up of patch[1], which makes call sites of
|Initialize()| to just one.

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

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

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

[commit-bot: I haz the power, 2017-05-31 08:56:17]

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