Revert of Improve fallback for Burmese with leading punctuation + spacing mark

Revert of Improve fallback for Burmese with leading punctuation + spacing mark (patchset #3 id:30009 of https://codereview.chromium.org/2530153002/ )

Reason for revert:
This seems to be causing test failures for fast/text/international/text-spliced-font.html on https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win10/builds/17622

Original issue's description:
> Improve fallback for Burmese with leading punctuation + spacing mark
>
> Issue 618178 describes an example where a Burmese text run starts with a
> leading punctuation character followed by a combining spacing mark. This
> grapheme cannot be shaped with the default font, since Times for example
> cannot display the combination of a left quote with a Burmese combining
> mark. Our fallback code attempts to find a fallback font based on the
> first character at the beginning of an extracted unshaped sub-run, which
> does not lead to finding a font suitable for Myanmar text in this case.
> So in a way it runs into a fallback trap, where no fallback hint helps
> to find the right fallback font and the whole run ends up as notdef
> glyphs.
>
> This CL attempts to resolve this by looking for a better fallback hint
> character, which is not script common or inherited, if such is
> available. This improves the situation for the Burmese text from the
> issue report.
>
> In addition, as a better fix we should give higher importance to the
> locale information in font fallback, filed as issue 668706.
>
> BUG=618178
> R=eae,kojii,behdad
>
> Committed: https://crrev.com/d9280a5e951415e1b2c6c7958adda19b731a99fa
> Cr-Commit-Position: refs/heads/master@{#434665}

TBR=behdad@chromium.org,eae@chromium.org,kojii@chromium.org,drott@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=618178
Committed: https://crrev.com/1b8f4236e2dbf4e07bf0171300003168633b8660
Cr-Commit-Position: refs/heads/master@{#434719}

[Marijn Kruisselbrink, 2016-11-28 21:06:54]

The CQ bit was checked by mek@chromium.org

[Marijn Kruisselbrink, 2016-11-28 21:06:54]

Created Revert of Improve fallback for Burmese with leading punctuation +
spacing mark

[commit-bot: I haz the power, 2016-11-28 21:07:56]

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-28 21:08:45]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480367214486550, "parent_rev":
"c35e3a0297dcc42af154777419a4935e8a98e839", "commit_rev":
"4877c6457ce1726c4ec40b024c534069f8fc0bbe"}

[commit-bot: I haz the power, 2016-11-28 21:09:12]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-28 21:13:00]

Description was changed from

==========
Revert of Improve fallback for Burmese with leading punctuation + spacing mark
(patchset #3 id:30009 of https://codereview.chromium.org/2530153002/ )

Reason for revert:
This seems to be causing test failures for
fast/text/international/text-spliced-font.html on
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win10/builds/1...

Original issue's description:
> Improve fallback for Burmese with leading punctuation + spacing mark
> 
> Issue 618178 describes an example where a Burmese text run starts with a
> leading punctuation character followed by a combining spacing mark. This
> grapheme cannot be shaped with the default font, since Times for example
> cannot display the combination of a left quote with a Burmese combining
> mark. Our fallback code attempts to find a fallback font based on the
> first character at the beginning of an extracted unshaped sub-run, which
> does not lead to finding a font suitable for Myanmar text in this case.
> So in a way it runs into a fallback trap, where no fallback hint helps
> to find the right fallback font and the whole run ends up as notdef
> glyphs.
> 
> This CL attempts to resolve this by looking for a better fallback hint
> character, which is not script common or inherited, if such is
> available. This improves the situation for the Burmese text from the
> issue report.
> 
> In addition, as a better fix we should give higher importance to the
> locale information in font fallback, filed as issue 668706.
> 
> BUG=618178
> R=eae,kojii,behdad
> 
> Committed: https://crrev.com/d9280a5e951415e1b2c6c7958adda19b731a99fa
> Cr-Commit-Position: refs/heads/master@{#434665}

TBR=behdad@chromium.org,eae@chromium.org,kojii@chromium.org,drott@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=618178
==========

to

==========
Revert of Improve fallback for Burmese with leading punctuation + spacing mark
(patchset #3 id:30009 of https://codereview.chromium.org/2530153002/ )

Reason for revert:
This seems to be causing test failures for
fast/text/international/text-spliced-font.html on
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Win10/builds/1...

Original issue's description:
> Improve fallback for Burmese with leading punctuation + spacing mark
>
> Issue 618178 describes an example where a Burmese text run starts with a
> leading punctuation character followed by a combining spacing mark. This
> grapheme cannot be shaped with the default font, since Times for example
> cannot display the combination of a left quote with a Burmese combining
> mark. Our fallback code attempts to find a fallback font based on the
> first character at the beginning of an extracted unshaped sub-run, which
> does not lead to finding a font suitable for Myanmar text in this case.
> So in a way it runs into a fallback trap, where no fallback hint helps
> to find the right fallback font and the whole run ends up as notdef
> glyphs.
>
> This CL attempts to resolve this by looking for a better fallback hint
> character, which is not script common or inherited, if such is
> available. This improves the situation for the Burmese text from the
> issue report.
>
> In addition, as a better fix we should give higher importance to the
> locale information in font fallback, filed as issue 668706.
>
> BUG=618178
> R=eae,kojii,behdad
>
> Committed: https://crrev.com/d9280a5e951415e1b2c6c7958adda19b731a99fa
> Cr-Commit-Position: refs/heads/master@{#434665}

TBR=behdad@chromium.org,eae@chromium.org,kojii@chromium.org,drott@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=618178

Committed: https://crrev.com/1b8f4236e2dbf4e07bf0171300003168633b8660
Cr-Commit-Position: refs/heads/master@{#434719}
==========

[commit-bot: I haz the power, 2016-11-28 21:13:01]

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