Integrate SymbolsIterator into RunSegmenter

Integrate SymbolsIterator into RunSegmenter

RunSegmenter is our API for segmenting TextRuns into sub-runs that
HarfBuzz can shape. By adding the SymbolsIterator to this segmentation
we gain control over emoji presentation type: text oder emoji. And we
can pre-select fonts that work better for symbols, math, punctuation.

BUG=580074
R=eae,kojii,behdad

Committed: https://crrev.com/8d0aeb2dced892bda385668da1fae29236e858b5
Cr-Commit-Position: refs/heads/master@{#372722}

[drott, 2016-01-21 15:27:10]

Depends on https://codereview.chromium.org/1613153002/# and it's probably better
to finalize the review after that one lands, but it may serve as a bit of a
preview and an initial review may start from my point of view.

[eae, 2016-01-22 00:39:27]

What are the run-time performance implications of adding yet another iterator?

[behdad, 2016-01-22 13:22:17]

lgtm

[drott, 2016-01-22 14:02:06]

On 2016/01/22 00:39:27, eae wrote:
> What are the run-time performance implications of adding yet another iterator?

Please see: https://code.google.com/p/chromium/issues/detail?id=580074#c2

[eae, 2016-02-01 06:58:00]

LGTM

[drott, 2016-02-01 08:44:56]

The CQ bit was checked by drott@chromium.org

[commit-bot: I haz the power, 2016-02-01 08:45:18]

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

[commit-bot: I haz the power, 2016-02-01 10:39:51]

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

[commit-bot: I haz the power, 2016-02-01 10:39:52]

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-02-01 11:07:18]

The CQ bit was checked by drott@chromium.org

[commit-bot: I haz the power, 2016-02-01 11:07:37]

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

[commit-bot: I haz the power, 2016-02-01 13:04:43]

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

[commit-bot: I haz the power, 2016-02-01 13:04:43]

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-02-01 18:27:41]

Rebaseline likely due to different run segmentation

[drott, 2016-02-01 18:27:48]

The CQ bit was checked by drott@chromium.org

[drott, 2016-02-01 18:27:49]

The patchset sent to the CQ was uploaded after l-g-t-m from eae@chromium.org,
behdad@chromium.org
Link to the patchset: https://codereview.chromium.org/1613903003/#ps20001
(title: "Rebaseline likely due to different run segmentation")

[commit-bot: I haz the power, 2016-02-01 18:31:11]

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

[commit-bot: I haz the power, 2016-02-01 19:41:10]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-01 19:42:19]

Description was changed from

==========
Integrate SymbolsIterator into RunSegmenter

RunSegmenter is our API for segmenting TextRuns into sub-runs that
HarfBuzz can shape. By adding the SymbolsIterator to this segmentation
we gain control over emoji presentation type: text oder emoji. And we
can pre-select fonts that work better for symbols, math, punctuation.

BUG=580074
R=eae,kojii,behdad
==========

to

==========
Integrate SymbolsIterator into RunSegmenter

RunSegmenter is our API for segmenting TextRuns into sub-runs that
HarfBuzz can shape. By adding the SymbolsIterator to this segmentation
we gain control over emoji presentation type: text oder emoji. And we
can pre-select fonts that work better for symbols, math, punctuation.

BUG=580074
R=eae,kojii,behdad

Committed: https://crrev.com/8d0aeb2dced892bda385668da1fae29236e858b5
Cr-Commit-Position: refs/heads/master@{#372722}
==========

[commit-bot: I haz the power, 2016-02-01 19:42:20]

Patchset 2 (id:??) landed as
https://crrev.com/8d0aeb2dced892bda385668da1fae29236e858b5
Cr-Commit-Position: refs/heads/master@{#372722}