Wrap code into TextIterator::HandleRememberedProgress

Wrap code into TextIterator::HandleRememberedProgress

TextIterator::Advance() used to contain some code before its main loop
to handle the remembered progress, if any. This patch wraps the code
into a new function TextIterator::HandleRememberedProgress() for better
code health.

Follow-up patches will utilize this function to make the control flow
in TextIterator cleaner.

BUG=721957
TEST=n/a; no behavioral change

Review-Url: https://codereview.chromium.org/2911373003
Cr-Commit-Position: refs/heads/master@{#475772}
Committed: https://chromium.googlesource.com/chromium/src/+/4ac11ed7d02420bda817ec1441af56f2b56a0a02

[Xiaocheng, 2017-05-31 01:28:01]

Description was changed from

==========
Wrap code into TextIterator::HandleRememberedProgress

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

to

==========
Wrap code into TextIterator::HandleRememberedProgress
BUG=721957
TEST=n/a; no behavioral change
==========

[Xiaocheng, 2017-05-31 01:28:04]

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

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

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-31 01:30:06]

Description was changed from

==========
Wrap code into TextIterator::HandleRememberedProgress
BUG=721957
TEST=n/a; no behavioral change
==========

to

==========
Wrap code into TextIterator::HandleRememberedProgress

TextIterator::Advance() used to contain some code before its main loop
to handle the remembered progress, if any. This patch wraps the code
into a new function TextIterator::HandleRememberedProgress() for better
code health.

Follow-up patches will utilize this function to make the control flow
in TextIterator cleaner.

BUG=721957
TEST=n/a; no behavioral change
==========

[Xiaocheng, 2017-05-31 01:30:14]

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

[Xiaocheng, 2017-05-31 01:30:14]

PTAL.

[yosin_UTC9, 2017-05-31 01:54:40]

The CQ bit was unchecked by yosin@chromium.org

[yosin_UTC9, 2017-05-31 01:54:43]

The CQ bit was checked by yosin@chromium.org

[yosin_UTC9, 2017-05-31 01:54:44]

lgtm

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

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 03:35:18]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1496195683497770, "parent_rev":
"1e511b111bdebc1cdb7588d9c22e034ab70715b9", "commit_rev":
"4ac11ed7d02420bda817ec1441af56f2b56a0a02"}

[commit-bot: I haz the power, 2017-05-31 03:35:28]

Description was changed from

==========
Wrap code into TextIterator::HandleRememberedProgress

TextIterator::Advance() used to contain some code before its main loop
to handle the remembered progress, if any. This patch wraps the code
into a new function TextIterator::HandleRememberedProgress() for better
code health.

Follow-up patches will utilize this function to make the control flow
in TextIterator cleaner.

BUG=721957
TEST=n/a; no behavioral change
==========

to

==========
Wrap code into TextIterator::HandleRememberedProgress

TextIterator::Advance() used to contain some code before its main loop
to handle the remembered progress, if any. This patch wraps the code
into a new function TextIterator::HandleRememberedProgress() for better
code health.

Follow-up patches will utilize this function to make the control flow
in TextIterator cleaner.

BUG=721957
TEST=n/a; no behavioral change

Review-Url: https://codereview.chromium.org/2911373003
Cr-Commit-Position: refs/heads/master@{#475772}
Committed:
https://chromium.googlesource.com/chromium/src/+/4ac11ed7d02420bda817ec1441af...
==========

[commit-bot: I haz the power, 2017-05-31 03:35:29]

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