Remove isBreakValid for Emoji sequences

Remove isBreakValid for Emoji sequences

isBreakValid was added to avoid wrap lines in the middle of an Emoji
sequence (ZWJ-linked or Emoji base + Emoji modifier), but it's not
necessary any more now that ICU was updated to 58.1 that handles
Emoji sequences correctly.

BUG=593263
TEST=blink_platform_unittest --gtest_filter=*BreakItera*

Committed: https://crrev.com/62d66e352097e36b6f7846b68f9e0b05eb9598ad
Cr-Commit-Position: refs/heads/master@{#430691}

[jungshik at Google, 2016-11-05 09:11:39]

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

[commit-bot: I haz the power, 2016-11-05 09:11:44]

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

[jungshik at Google, 2016-11-05 09:12:06]

jshin@chromium.org changed reviewers:
+ drott@chromium.org

[jungshik at Google, 2016-11-05 09:12:07]

Dominik, can you take a look? Thanks

[commit-bot: I haz the power, 2016-11-05 10:32:32]

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

[commit-bot: I haz the power, 2016-11-05 10:32:33]

Dry run: This issue passed the CQ dry run.

[drott, 2016-11-08 09:12:55]

Fantastic, so happy we can get rid of this! LGTM assuming that the test strings
are exactly identical to before, just encoded differently.

[drott, 2016-11-08 09:13:15]

Thanks for taking care of this issue.

[jungshik at Google, 2016-11-08 18:17:45]

On 2016/11/08 09:12:55, drott wrote:
> Fantastic, so happy we can get rid of this! LGTM assuming that the test
strings
> are exactly identical to before, just encoded differently.

Thanks for the review. Just in case, I checked once again that test strings have
not changed (just encoded differently).

[jungshik at Google, 2016-11-08 18:17:50]

The CQ bit was checked by jshin@chromium.org

[commit-bot: I haz the power, 2016-11-08 18:18:14]

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-11-08 19:55:48]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-08 20:10:48]

Description was changed from

==========
Remove isBreakValid for Emoji sequences

isBreakValid was added to avoid wrap lines in the middle of an Emoji
sequence (ZWJ-linked or Emoji base + Emoji modifier), but it's not
necessary any more now that ICU was updated to 58.1 that handles
Emoji sequences correctly.

BUG=593263
TEST=blink_platform_unittest --gtest_filter=*BreakItera*
==========

to

==========
Remove isBreakValid for Emoji sequences

isBreakValid was added to avoid wrap lines in the middle of an Emoji
sequence (ZWJ-linked or Emoji base + Emoji modifier), but it's not
necessary any more now that ICU was updated to 58.1 that handles
Emoji sequences correctly.

BUG=593263
TEST=blink_platform_unittest --gtest_filter=*BreakItera*

Committed: https://crrev.com/62d66e352097e36b6f7846b68f9e0b05eb9598ad
Cr-Commit-Position: refs/heads/master@{#430691}
==========

[commit-bot: I haz the power, 2016-11-08 20:10:49]

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