Force expandEnd/RangeToSentenceBoundary to return a valid EphemeralRange

Force expandEnd/RangeToSentenceBoundary to return a valid EphemeralRange

This is a first-aid patch that make expandEnd/RangeToSentenceBoundary()
compare the sentence boundaries found with the input range before
returning, so that the returned range is always a super-range of
the input range, and hence, a valid EphemeralRange.

This patch does not fix the root cause of the bugs, as we do not expect startOfSentence()'s return value to be beyond that of endOfSentence()'s.
It simply stops the crashing at expandEnd/RangeToSentenceBoundary.

BUG=640905
TEST=n/a; this is a first-aid patch

Committed: https://crrev.com/5345d9cf9d3d0ad37cb56edd31aa0686d3ff22c5
Cr-Commit-Position: refs/heads/master@{#414624}

[Xiaocheng, 2016-08-25 10:00:13]

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

[commit-bot: I haz the power, 2016-08-25 10:00:37]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Xiaocheng, 2016-08-25 10:04:22]

Description was changed from

==========
Force expandEnd/RangeToSentenceBoundaries to return valid EphemeralRange

BUG=640905
TEST=n/a; this is a first-aid patch
==========

to

==========
Force expandEnd/RangeToSentenceBoundary to return a valid EphemeralRange

This is a first-aid patch that make expandEnd/RangeToSentenceBoundary()
compare the sentence boundaries found with the input range before
returning, so that the returned range is always a super-range of
the input range, and hence, a valid EphemeralRange.

This patch does not fix the root cause of the bugs, as we do not expect
startOfSentence()'s return value to be beyond that of endOfSentence()'s.
It simply stops the crashing at expandEnd/RangeToSentenceBoundary.

BUG=640905
TEST=n/a; this is a first-aid patch
==========

[Xiaocheng, 2016-08-25 10:04:31]

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

[Xiaocheng, 2016-08-25 10:04:32]

PTAL.

[commit-bot: I haz the power, 2016-08-25 13:11:35]

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

[commit-bot: I haz the power, 2016-08-25 13:11:36]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[Xiaocheng, 2016-08-25 13:16:38]

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

[commit-bot: I haz the power, 2016-08-25 13:16:58]

Dry run: 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, 2016-08-25 14:51:55]

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

[commit-bot: I haz the power, 2016-08-25 14:51:55]

Dry run: This issue passed the CQ dry run.

[yosin_UTC9, 2016-08-26 01:35:14]

lgtm

We'll fix boundary related calculation next quarter...

[Xiaocheng, 2016-08-26 01:41:04]

The CQ bit was checked by xiaochengh@chromium.org

[commit-bot: I haz the power, 2016-08-26 01:41:33]

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, 2016-08-26 01:46:35]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-26 01:49:16]

Description was changed from

==========
Force expandEnd/RangeToSentenceBoundary to return a valid EphemeralRange

This is a first-aid patch that make expandEnd/RangeToSentenceBoundary()
compare the sentence boundaries found with the input range before
returning, so that the returned range is always a super-range of
the input range, and hence, a valid EphemeralRange.

This patch does not fix the root cause of the bugs, as we do not expect
startOfSentence()'s return value to be beyond that of endOfSentence()'s.
It simply stops the crashing at expandEnd/RangeToSentenceBoundary.

BUG=640905
TEST=n/a; this is a first-aid patch
==========

to

==========
Force expandEnd/RangeToSentenceBoundary to return a valid EphemeralRange

This is a first-aid patch that make expandEnd/RangeToSentenceBoundary()
compare the sentence boundaries found with the input range before
returning, so that the returned range is always a super-range of
the input range, and hence, a valid EphemeralRange.

This patch does not fix the root cause of the bugs, as we do not expect
startOfSentence()'s return value to be beyond that of endOfSentence()'s.
It simply stops the crashing at expandEnd/RangeToSentenceBoundary.

BUG=640905
TEST=n/a; this is a first-aid patch

Committed: https://crrev.com/5345d9cf9d3d0ad37cb56edd31aa0686d3ff22c5
Cr-Commit-Position: refs/heads/master@{#414624}
==========

[commit-bot: I haz the power, 2016-08-26 01:49:16]

Patchset 1 (id:??) landed as
https://crrev.com/5345d9cf9d3d0ad37cb56edd31aa0686d3ff22c5
Cr-Commit-Position: refs/heads/master@{#414624}