SegmentedString: Use decrementAndCheckLength where possible

SegmentedString: Use decrementAndCheckLength where possible

This CL removes duplicated decrementAndCheckLength() logic in SegmentedString.

BUG=None
TESTS=Refactoring only. No change in behaviour

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=201286

[kouhei (in TOK), 2015-08-27 03:01:23]

kouhei@chromium.org changed reviewers:
+ tkent@chromium.org

[tkent, 2015-08-27 03:08:01]

lgtm

https://codereview.chromium.org/1320813002/diff/1/Source/platform/text/Segmen...
File Source/platform/text/SegmentedString.h (right):

https://codereview.chromium.org/1320813002/diff/1/Source/platform/text/Segmen...
Source/platform/text/SegmentedString.h:238: if (haveNewLine) {
This block doesn't refer to data members updated by
updateSlowCaseFunctionPointers().  So, the change should be safe.

[kouhei (in TOK), 2015-08-27 03:08:35]

The CQ bit was checked by kouhei@chromium.org

[commit-bot: I haz the power, 2015-08-27 03:08:46]

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

[blink-reviews, 2015-08-27 03:09:44]

Thanks!

On Thu, Aug 27, 2015 at 12:08 PM, commit-bot@chromium.org via
codereview.chromium.org <reply@chromiumcodereview-hr.appspotmail.com> wrote:

> CQ is trying da patch. Follow status at
>  https://chromium-cq-status.appspot.com/patch-status/1320813002/1
> View timeline at
>  https://chromium-cq-status.appspot.com/patch-timeline/1320813002/1
>
>
> https://codereview.chromium.org/1320813002/
>



-- 
Kouhei Ueno

To unsubscribe from this group and stop receiving emails from it, send an email
to blink-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2015-08-27 05:05:46]

Committed patchset #1 (id:1) as
https://src.chromium.org/viewvc/blink?view=rev&revision=201286