During recorrection, the cursor position when calling
commitText is not necessarily at the end of the
composing text.
Besides, RichInputConnection assumes the cursor is
always after any composing text. This is not correct,
but in the practice, it seems all code paths work.
We should fix this in the future.

Bug: 13060691
Change-Id: I15f71fff62d36e80cf6e4a022c5e78af634b199d

Bug: 12483313
Change-Id: Ia811d614b9260367871bc5717f587538adf46789

Change-Id: I63a20bb2e4b3c39495fade6cae3a14fdd11ca48c

This old method doesn't even re-read the old suggestions. It used to
recompute them without the coordinates.
Re-using the recorrection code, which is much more advanced, is
the right thing to do here.

Also, refining the test. It's no use trying to resume suggestion
if we don't have a suggestion strip, since we aren't going to
auto-correct anything anyway.

Not the motivation for this change, but this also fixes
Bug: 11620256
Change-Id: Id49efa32e293c49837c61fdc752c86bbac1d2c88

Bug: 12295276
Change-Id: Id359c92acde44758b12929e7bac719d5c9c7577b

Change-Id: I056013de3a5e09feed429dfd8097d3d85331a0ab

This also includes a fix that allows this code to read surrogate
pairs in this processing.

Bug: 11070482
Change-Id: If5ef8d6863938252f09128b7e99ea07ece6e7019

Bug: 10445496
Change-Id: Ib7bd05aea59ae6c9b8ce31299ef2235521b0e350

Bug: 10403176
Change-Id: I510bf1dccf6795426e6fb1b36181b18039a0f435

Thai is an example.

Bug: 10138062
Change-Id: Ib9a8264c77ed42b4256432d7c8a60d08575dcdc7

Bug: 8911898
Change-Id: Iede69ec2d9870df0add5ab818613fbb8e1f943c2

Bug: 8613307

Change-Id: Ief04a4abb6dc1f79660a2e419c9c5e271802b5a1

Change-Id: I32700c434b296bb3fd39e040c2dda1fe90667daf

Bug: 8131968
Change-Id: Ibca5a0d63a492134b8af401a62ca3a5748e003cf

And add a test to make sure it stays not-broken.

Bug: 7946604
Change-Id: I996da3d5507d591ec25a13fb57434f39843f1df5

Change-Id: Iee01d4d2b916d0b584531104ac865ae6e6370a3d

Most of the failures can be ascribed to the tests not passing the correct
old position of the cursor on a second call to onUpdateSelection() to
LatinIME.

Bug: 7276565
Bug: 7276805
Bug: 7276195
Change-Id: I3f1b52cdcc783ea18838408bed50699b7254eaf4

Bug: 6890540
Change-Id: I923b24e3bc17e7b923be4940ad739b58dca24a1f

We stopped cancelling manual picks a few weeks ago. This code
is dead.

Change-Id: I4032fcc3c95e9379f1839fe860a1b8a9bd7d0bc7

This behavior has been changed for bug#6105732

Bug: 6516976
Change-Id: Ief8225b26d831cd92a051ce25cc005270b6b1776

Bug: 6188932
Change-Id: Ibcc4901bcfab6632ee4c59cb58d35452218a288d

The intent is to make things easier to read.

Change-Id: Ida95ac8bd4119a2497d3e2a4d92485a96316962d

This will allow easier reuse of the complicated set up code
and the various helper methods.

Change-Id: I864eb0da476cbf0fe087ddb1713af80d3c3eb850

This fixes the input logic tests that were broken and suppresses
their dependency upon the spell checker proximity. Instead, it
gets the Keyboard instance from Latin IME and uses the actual
coordinates, which results in a test run closer to what actually
happens during typing.

Change-Id: I3a81d249ee7fb3ac6ae6940aa2e8b2421e829e5c

Bug: 6174065
Change-Id: I702760d44ead0eeb60d06360aa3bb03c2ec73325

Also fix a typo, and increase the time allowed to load the main
dictionary to 2 secs instead of 1.

Bug: 6114326
Change-Id: I5f70b34fc4277c55977b18466253152aa98a2507

Bug: 6121571
Change-Id: Ib16fbef1087da3b7da5539901ec57156f242f089

Bug: 4343280

Change-Id: I0f6a7e6912ed4abea07c10d266da4c7ccb0dae76

To some extent, the test was wrong here.
After space is typed in this case, we wait a small delay before updating
the suggestion strip so that we can display bigram predictions without
introducing delays. The suggestion strip is not displayed until then.

The test was bypassing this by simulating a press on the suggestion
strip, but for LatinIME it was not displayed yet, hence the bug.
The new code waits for the delay, makes sure the suggestion strip is
actually displaying punctuation marks, then press the mark and do
the intended test on space.

Change-Id: I18f82da1bc0a8e584e9b1bac59ff48823d78ec12

Bug: 5975557
Change-Id: I39e782f89813b9d585e369e590d6288e696168a2

This is really brutal, but after trying many many things I didn't
find a better way. It works with the current implementation of
MessageQueue.
Since this only introduces a utility method it doesn't really do
anything, but the functionality it provides will be essential to
Bug: 5975557

Change-Id: I81c4113a08f9a8d8a88294d7dd3b6c8c483c8b1d

- Type "tgis", manually pick "thus", press backspace.
  Backspace should revert the manual pick.
  Check "tgis" is the result.
- Type "tgis" followed by a period.
  Period should trigger auto-correct to "this".
  Check "this" is the result.
- Type "tgis" followed by a period, then backspace.
  Period should trigger auto-correct to "this", and backspace should
  revert the auto-correction.
  Check "tgis." is the result.

Change-Id: I7e23c8a26fbdbe23336149a05ff01bc51707422e

Change-Id: I61ff218ae2ca4eb443a370e581b677755258670a

Bug: 5962053
Change-Id: I7dd68767be6e724675428910fa2c00290f76e869

Change-Id: Ic19d380eb64855884bc6ded8547c200dd772ffe9

Bug: 4994861
Change-Id: I6ae256a94dbaddea4304304779d17025620c5025

And unit test

Change-Id: Ic4fc3626f8b86e10156d770d41cd6deab5d31f39

Seems I didn't get how to iterate on a String correctly >.>
Talk about a big bug. Anyway, I think it's working now.

Bug: 5955228
Change-Id: I988c900cf2a16c44b9505cfd4f77c7cda7e592f0

Bug: 4994861
Change-Id: I114a1117cf19aa6e514ac7342f733175dc5d5df1

Change-Id: Ia51da6506d78a295b8b70a241208fb81602cf0ea