Enable Emoji Fitzpatrick modifier sequences

Enable Emoji Fitzpatrick modifier sequences

* The CJK segmentation step in CachingWordShapeIterator should keep
  modifier sequences intact: Missing emoji need to be added to our
  CharacterData tables as well as allowing modifier characters as a
  continuation of CJK sequences in the nextWordEndIndex() step.
* Disable blanking emoji modifier sequences in HarfBuzzShaper
* Upgrade text code path from simple to complex for modifier characters,
  required for Android until Android switches to complex by default

BUG=476377
R=kojii,eae

Committed: https://crrev.com/184a32d58946f709f6f8c5b6037a772431be5530
Cr-Commit-Position: refs/heads/master@{#380399}

[drott, 2016-03-09 19:21:54]

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

[commit-bot: I haz the power, 2016-03-09 19:22:48]

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

[eae, 2016-03-09 19:33:45]

LGTM

https://codereview.chromium.org/1780723002/diff/1/third_party/WebKit/Source/p...
File third_party/WebKit/Source/platform/fonts/Character.cpp (right):

https://codereview.chromium.org/1780723002/diff/1/third_party/WebKit/Source/p...
third_party/WebKit/Source/platform/fonts/Character.cpp:201: if
(supplementaryCharacter < 0x1F3FB)
Could we add named constants or a comment to explain the codepoints?

[commit-bot: I haz the power, 2016-03-09 20:36:30]

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

[commit-bot: I haz the power, 2016-03-09 20:36:31]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[drott, 2016-03-10 07:45:38]

Comment and comparison fix for codepath choice

[drott, 2016-03-10 07:46:02]

The CQ bit was checked by drott@chromium.org

[drott, 2016-03-10 07:46:02]

The patchset sent to the CQ was uploaded after l-g-t-m from eae@chromium.org
Link to the patchset: https://codereview.chromium.org/1780723002/#ps20001
(title: "Comment and comparison fix for codepath choice")

[commit-bot: I haz the power, 2016-03-10 07:46:20]

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

[commit-bot: I haz the power, 2016-03-10 07:48:28]

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

[commit-bot: I haz the power, 2016-03-10 07:48:30]

Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[drott, 2016-03-10 09:26:00]

rebased on renamed CharacterData file

[drott, 2016-03-10 09:26:06]

The CQ bit was checked by drott@chromium.org

[drott, 2016-03-10 09:26:06]

The patchset sent to the CQ was uploaded after l-g-t-m from eae@chromium.org
Link to the patchset: https://codereview.chromium.org/1780723002/#ps40001
(title: "rebased on renamed CharacterData file")

[commit-bot: I haz the power, 2016-03-10 09:26:14]

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

[commit-bot: I haz the power, 2016-03-10 13:29:15]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-10 13:31:27]

Description was changed from

==========
Enable Emoji Fitzpatrick modifier sequences

* The CJK segmentation step in CachingWordShapeIterator should keep
  modifier sequences intact: Missing emoji need to be added to our
  CharacterData tables as well as allowing modifier characters as a
  continuation of CJK sequences in the nextWordEndIndex() step.
* Disable blanking emoji modifier sequences in HarfBuzzShaper
* Upgrade text code path from simple to complex for modifier characters,
  required for Android until Android switches to complex by default

BUG=476377
R=kojii,eae
==========

to

==========
Enable Emoji Fitzpatrick modifier sequences

* The CJK segmentation step in CachingWordShapeIterator should keep
  modifier sequences intact: Missing emoji need to be added to our
  CharacterData tables as well as allowing modifier characters as a
  continuation of CJK sequences in the nextWordEndIndex() step.
* Disable blanking emoji modifier sequences in HarfBuzzShaper
* Upgrade text code path from simple to complex for modifier characters,
  required for Android until Android switches to complex by default

BUG=476377
R=kojii,eae

Committed: https://crrev.com/184a32d58946f709f6f8c5b6037a772431be5530
Cr-Commit-Position: refs/heads/master@{#380399}
==========

[commit-bot: I haz the power, 2016-03-10 13:31:28]

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