Refactor and comment BreakingContext::handleText() - Part 2

Refactor and comment BreakingContext::handleText() - Part 2

Continue the refactoring from https://codereview.chromium.org/1504083002
by moving most of the detail in this function to helper functions. This
makes the behaviour of the function more explicit - namely a loop that
skips over non-whitespace and/or collapsible whitespace but returns early
if it encounters a soft or hard line-breaking opportunity.

BUG=567080
Committed: https://crrev.com/0b36cf70d82e790bf6e1fe62a35b3705f702a2ed
Cr-Commit-Position: refs/heads/master@{#367052}

[rhogan, 2015-12-16 19:03:59]

Description was changed from

==========
Refactor and comment BreakingContext::handleText()

BUG=567080
==========

to

==========
Refactor and comment BreakingContext::handleText() - Part 2

Continue the refactoring from https://codereview.chromium.org/1504083002
by moving most of the detail in this function to helper functions. This
makes the behaviour of the function more explicit - namely a loop that
skips over non-whitespace and/or collapsible whitespace but returns early
if it encounters a soft or hard line-breaking opportunity.

BUG=567080
==========

[rhogan, 2015-12-17 20:25:18]

robhogan@gmail.com changed reviewers:
+ eae@chromium.org, leviw@chromium.org

[rhogan, 2015-12-21 18:40:41]

robhogan@gmail.com changed reviewers:
- leviw@chromium.org

[rhogan, 2015-12-23 17:27:52]

On 2015/12/17 at 20:25:18, rhogan wrote:
>

[rhogan, 2015-12-23 17:28:21]

On 2015/12/23 at 17:27:52, rhogan wrote:
> On 2015/12/17 at 20:25:18, rhogan wrote:
> >

Hi there - eae can you take a look?

[eae, 2015-12-28 17:35:14]

Those function signatures makes my head hurt but it is a step in the right
direction, thank you for working on this.

LGTM

[rhogan, 2015-12-29 04:01:25]

The CQ bit was checked by robhogan@gmail.com

[commit-bot: I haz the power, 2015-12-29 04:02:46]

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

[commit-bot: I haz the power, 2015-12-29 04:42:34]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-12-29 04:43:35]

Description was changed from

==========
Refactor and comment BreakingContext::handleText() - Part 2

Continue the refactoring from https://codereview.chromium.org/1504083002
by moving most of the detail in this function to helper functions. This
makes the behaviour of the function more explicit - namely a loop that
skips over non-whitespace and/or collapsible whitespace but returns early
if it encounters a soft or hard line-breaking opportunity.

BUG=567080
==========

to

==========
Refactor and comment BreakingContext::handleText() - Part 2

Continue the refactoring from https://codereview.chromium.org/1504083002
by moving most of the detail in this function to helper functions. This
makes the behaviour of the function more explicit - namely a loop that
skips over non-whitespace and/or collapsible whitespace but returns early
if it encounters a soft or hard line-breaking opportunity.

BUG=567080

Committed: https://crrev.com/0b36cf70d82e790bf6e1fe62a35b3705f702a2ed
Cr-Commit-Position: refs/heads/master@{#367052}
==========

[commit-bot: I haz the power, 2015-12-29 04:43:35]

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