Thanks Eclipse

Change-Id: Ie2b83f1ec7ab38a76155f8c264e3944685ae934d

Change-Id: I0578068946d32192c4faf14b3e86624b7301c7d9

It's still incorrect for non-English languages.

Bug: 4967874
Change-Id: I3d4379265f72e1c0435eba90ab878a80bbcf55ce

Change-Id: I6248f9f3ff99003ab62be22eac673df0f7f5c015

Change-Id: Id3ba200e06244e6d13cf57d346001d61f85ef1a2

Change-Id: I1b1763ceefda6357dc698195928d0f16bc425830

Change-Id: I2ea9566ce5cb1fbccd33a42836244a899365de6a

Since the function has to be modified heavily but does a lot
of non-trivial work, add a wealth of comments explaining what
it does and why so as to facilitate understanding the changes
to come.

Bug: 4967874
Change-Id: I6c21aea15f161d807035f279dfb7d1b98b3e9144

This should have on effect at all on behavior,
except an increase in performance.

Bug: 4967874
Bug: 6950087
Change-Id: Ie2b51efefe84ca767f5dc8e3b80bfef7e1faab3d

Change-Id: Ic4f69b4c8da33c9fca50d6829724179310c2f875

Change-Id: I6b4de9187e122298e5e9cd8ddc9070d062df6a89

StringUtils.toCodePointArray() had thrown IllegalOutOfBoundsException if
passed an empty string.  change to just return an empty int[].

Bug: 6188932
Change-Id: Ic41c628c0d407f49fc98cd48cb7ea13d8d5bdd77

Bug: 6363029
Change-Id: I71a1ed5b88f54fcc15bfaf89d3645b8f17ebb821

This change also refactor StringUtils class

Change-Id: Ie0b4d169b21c260bf238d6fcc9ab0ee8bfd6b508

Change-Id: I477b631d81ef58461e44954f3ae5fd895928bb97

Change-Id: Ic7c1ec0e0606da6eb5b954934547c6e8a561fed5

Change-Id: Ice1f345a08a8d760e3b847c885c4072e3e142c97

Bug: 6010147
Change-Id: I401c2e3fcd639c0e1a03e64489a0d792810caa18

The intention may have been nice originally but these end up
being copied anyway :/
Let's remove them now, and in a later change, just keep
references to the created objects.

Change-Id: Ifba8357c20384f9eb40cd916665ed1fc6dc8cab1

Change-Id: I09e91675fe7d573dad8c933ad513b21d7e409144