This is partial revert of Iced1349e3ca750821d8882c551551f65bb569794.

Due to sorting of target family vectors, the font family order from XML
settings file is broken. Making unique operation stable doesn't fix the
issue completely since some font families are appended for the fallback
which also breaks the original order.

By this change, itemization becomes 3x slower than before if variation
selector is appended.

Bug: 29585939
Change-Id: I7c1a8a57f04111a30cd41a5cd5bec25fcfb3972e

Fix google-explicit-constructor warnings in minikin am: 0114c6a1  -s ours am: 8402f2ee am: 593cccab
am: e189ad95

Change-Id: Ie330bd0bfec81c3a8d159a13d96480dbd2774855

am: 593cccab

Change-Id: I3623a0f372da99c5a189bc5900b3f55bc3d7b0be

am: 8402f2ee

Change-Id: Idf3e3882894774d917166e4eec743fb8bea0c31b

am: 0114c6a1  -s ours

Change-Id: I51c71217e402c3cbeefd6de5ccf7252dc1687f76

* Add explicit keyword to conversion constructors,
  or add NOLINT(implicit) for implicit converters.
Bug: 28341362
Test: build with WITH_TIDY=1

Change-Id: I0c7b90f9bb953a9f2e4f0fb2032fa65ac604b9ca
Merged-In: I0c7b90f9bb953a9f2e4f0fb2032fa65ac604b9ca

* Add explicit keyword to conversion constructors,
  or add NOLINT(implicit) for implicit converters.
Bug: 28341362
Test: build with WITH_TIDY=1

Change-Id: I0c7b90f9bb953a9f2e4f0fb2032fa65ac604b9ca

am: f83d57f7

Change-Id: Iebf945839891cc1ab561cf424ef917ac15e08842

am: de728a34

Change-Id: Id1b94e72ccd480bf5395752868f7f26e17ab7491

am: 1aa0c40d

Change-Id: Icc8b76ebd3edfdeb7622bf21a6585b191cf5b15f

This reverts commit 9431bb96.

Bug: 30815709
Change-Id: I057d9bcd05246e58894abb4e9633bd10f6fab211

Skia is moving to returning smart pointers from its factory methods.
This updates uses of SkTypeface::CreateXXX to SkTypeface::MakeXXX and
generally updates use of SkTypeface to sk_sp. This will allow for the
removal of the SK_SUPPORT_LEGACY_TYPEFACE_PTR define.

Change-Id: If3e600c6cb86080576667bc77d427da4f6560afa

am: 9480fd77

Change-Id: I6c77cba0290c56995bf56cfb876dd60989e50b9b

am: 249152f8  -s ours

Change-Id: Ib77b3862c81da7c58bd64846c0950534c12b5492

am: b746bef0  -s ours

Change-Id: I3fb287e86b3ecb8a61dca114578fe3a067a4469f

am: 9431bb96

Change-Id: Ic4900b65ed16a2ba3b5431d12a093654b7921a9b

am: c2642f9a

Change-Id: I551ee1c4c5368dde3bf2ab7d016634991b62f212

am: fab99dde

Change-Id: Ieda8b25bba40f22c9d8c33aba590e96e3396f6c4

Unicode recommends that the zwj sequences should be emoji presentation
even if they don't have the proper U+FE0F. Thus always lookup the glyph
for the code point before and after zwj as if they have a U+FE0F
variation selector.

Bug: 30056627
Change-Id: I03958a92337eaba4a8dd9c5be824b2665aa4a103

Bug: 29885295
Change-Id: I1bf191a46d05e7099265d863bae0523c50817d0b

FEMALE SIGN(U+2640), MALE SIGN(U+2642), StAFF OF AESCULAPIUS(U+2695)
will be used as the ZWJ sequenced in gender balanced emoji sequence.
To be in the same run with ZWJ, mark these emoji as sticky chracters.

With this fix, Female police officer sequence will be shown correctly
regardless of VS16.

Bug: 30026374
Change-Id: I503fc061eaa943d45208bb69e885151610c430ce

Unicode recommends that the zwj sequences should be emoji presentation
even if they don't have the proper U+FE0F. Thus always lookup the glyph
for the code point before and after zwj as if they have a U+FE0F
variation selector.

Bug: 30056627
Change-Id: I03958a92337eaba4a8dd9c5be824b2665aa4a103

Bug: 29885295
Change-Id: I1bf191a46d05e7099265d863bae0523c50817d0b

FEMALE SIGN(U+2640), MALE SIGN(U+2642), StAFF OF AESCULAPIUS(U+2695)
will be used as the ZWJ sequenced in gender balanced emoji sequence.
To be in the same run with ZWJ, mark these emoji as sticky chracters.

With this fix, Female police officer sequence will be shown correctly
regardless of VS16.

Bug: 30026374
Change-Id: I503fc061eaa943d45208bb69e885151610c430ce

stat.st_size is off_t not size_t, so need to cast to size_t before
compare it.

Change-Id: I6b742746fbb9f254701fc91e515c293f93f912c5

This CL introduces performance tests for FontCollection.

To support TTC file in /system/fonts, this CL also extends FontTestUtils

Bug:29142734
Change-Id: I9d8ad24ca55f61031b85623ab7c26234239e4f41

This CL introduces performance tests for following three modules:
- Hyphenator
- WordBreaker
- GraphemeBreak

During using UnicodeUtils, need to decouple it from gtest since
perftest doesn't have gtest dependencies.

Bug:29142734
Change-Id: I700c662fa7d0a52f19d8e93150ad1a85dc28769f

This fixes following three memory leaks in test utilities.
There is no problem in production code and this CL doesn't affect
any production behaviors.

- SkTypeface leaks due to forget calling SkSafeUnref in dtor.
- MinikinFontForTest leaks during constructing FontCollection.
- FontCollection leaks due to unnecessary AddRef.

Change-Id: I22e1e0307f1b2499296acb1aacc3ef66076a36e9

Here is a new policy of the namespace of minikin.
- All components should be in minikin namespace.
- All tests are also in minikin namespace and no anonymous namespace.

Bug: 29233740
Change-Id: I71a8a35049bb8d624f7a78797231e90fed1e2b8c

To add perftests and reuse some utility classes, reconstruct
test directory structure.

- Move unit tests from minikin/tests to minikin/tests/unittests
- Extract utilitiy classes to minikin/tests/utils which will be
  used by perftests eventually.

Change-Id: I5026b177934e72ae67d362ee888302037da2f808

Bug: 28992567
Change-Id: Id14cc86761d6c5b408262fe8684e2b293a420a4f

am: b11b991f

* commit 'b11b991f':

Change-Id: I0a3d24fcffb1d0533970058b4a3498d45e794f36

am: b4bbae32

* commit 'b4bbae32':
  Do not break after Myanmar viramas

Change-Id: I69972aed4fe1c1b956c0b65af18e1a4f1c739bf4

am: 8614b042

* commit '8614b042':

Change-Id: I4437e1206de2a45593e69956d8bcdf85a64b0f32

am: 00de2cc3

* commit '00de2cc3':
  Do not break after Myanmar viramas

Change-Id: I2e240fab84a49c73165056ee5ff18dbfca0faeec

am: 74b56175

* commit '74b56175':
  Do not break after Myanmar viramas

Change-Id: I13297120c3c3789c03b860c9e72b3a04922ab9fa

am: 74b56175

* commit '74b56175':
  Do not break after Myanmar viramas

Change-Id: Idb9303889ac87853a730cdb25fba7faaaf352b93

This is to work around a bug in ICU's line breaker, which thinks
there is a valid line break between a Myanmar kinzi and a consonant.
See http://bugs.icu-project.org/trac/ticket/12561 for the ICU bug.

Bug: 28964845
Change-Id: I076ac15077e5627cbccf6732900bcc60d8596dda