Detemplatize TextIterator::copyTextTo with ForwardsTextBuffer

Detemplatize TextIterator::copyTextTo with ForwardsTextBuffer

This is part of a bigger CL [1], which is in turn part of another even
bigger CL [2].

[1] https://codereview.chromium.org/1647483003
[2] https://codereview.chromium.org/1604783002

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

Committed: https://crrev.com/75b7224278e2e08df6ae31e3257e8cd91b40ef90
Cr-Commit-Position: refs/heads/master@{#373199}

[Xiaocheng, 2016-02-02 10:01:39]

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

[Xiaocheng, 2016-02-02 10:01:40]

PTAL.

[Xiaocheng, 2016-02-03 06:11:53]

Ping...

[yosin_UTC9, 2016-02-03 06:39:30]

I know it is too early to say so, but we would like to avoid ref-parameter.
Could you use |{Backwards,Forwards}TextBuffer*| for |copyTextTo()|?

[Xiaocheng, 2016-02-03 07:41:30]

On 2016/02/03 at 06:39:30, yosin wrote:
> I know it is too early to say so, but we would like to avoid ref-parameter.
> Could you use |{Backwards,Forwards}TextBuffer*| for |copyTextTo()|?

Done.

[yosin_UTC9, 2016-02-03 07:52:49]

lgtm

Thanks for simplification!

[yosin_UTC9, 2016-02-03 07:52:52]

The CQ bit was checked by yosin@chromium.org

[commit-bot: I haz the power, 2016-02-03 07:53:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1651263003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1651263003/40001

[tkent, 2016-02-03 08:02:32]

web/ lgtm

[commit-bot: I haz the power, 2016-02-03 08:03:32]

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

[commit-bot: I haz the power, 2016-02-03 08:03:33]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Xiaocheng, 2016-02-03 08:06:05]

The CQ bit was checked by xiaochengh@chromium.org

[commit-bot: I haz the power, 2016-02-03 08:06:19]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1651263003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1651263003/40001

[commit-bot: I haz the power, 2016-02-03 08:34:07]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-03 08:35:02]

Description was changed from

==========
Detemplatize TextIterator::copyTextTo with ForwardsTextBuffer

This is part of a bigger CL [1], which is in turn part of another even
bigger CL [2].

[1] https://codereview.chromium.org/1647483003
[2] https://codereview.chromium.org/1604783002

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

to

==========
Detemplatize TextIterator::copyTextTo with ForwardsTextBuffer

This is part of a bigger CL [1], which is in turn part of another even
bigger CL [2].

[1] https://codereview.chromium.org/1647483003
[2] https://codereview.chromium.org/1604783002

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

Committed: https://crrev.com/75b7224278e2e08df6ae31e3257e8cd91b40ef90
Cr-Commit-Position: refs/heads/master@{#373199}
==========

[commit-bot: I haz the power, 2016-02-03 08:35:03]

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