Change-Id: I5cfa0d2fccc139bd6c45c5590a68c3e0c90534b8

Change-Id: Icae6c74401a7a8f4da6015737f09f557640c0cca

Bug: 12179576
Change-Id: Ibafd824a0ea2251843b388e95dcbd8f9bca92d5c

This reverts commit 42e83c7f.

Bug: 12434783
Change-Id: Id01b6d14906430b13e64f5fc4d7fb01572d0a960

Change-Id: Ia0ef830a5bb3ed6fee1a5d113b2baec60bb51a97
Auto-generated-cl: translation import

Change-Id: I3e0560396e087669ff484b6c15d0f0eb03c02de5
Auto-generated-cl: translation import

Bug: 11736529
Change-Id: I0d9d9b1c1108b11064163df7c2089f53e92cd7ad

Bug: 12424891
Change-Id: If041d1ab45988dbf9b878ae3367fee0fbc8a8988

Don't use absolute cursor positions, this leads to race
conditions.

Bug: 12390573
Change-Id: I6f04f07a46babbc3d51cb94c7eaee21bed242396

Bug: 11933670
Change-Id: Ib0ce853e691bdadffb73874ea9f188cfbec8bc93

This change also fixes the background color of the left keys of the space key.

Bug: 12407208
Change-Id: Id869605146e9f72835f892c3c7ae14f118177394

Bug: 12377179
Change-Id: I44d35c6fa82883710674ecca6f630d09076472fe

Bug: 12377179
Change-Id: I4e2de78dc7e21f3f34eb56c790160b77cfc7584c

This change must be checked in together with I00b6e79d6b.

Change-Id: Id5f87a8d0286eb44958381fefd428f16896dc0d4

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: 8636060
Change-Id: Ifa027de465a421b67897d51e9ad47666dda117ad

...at last

Bug: 8636060
Change-Id: If5e27a9d28ff64d318dd76792f55edfc8a78d2d7

Bug: 8636060
Change-Id: Ie5027512f066d9c530a686ac559e4649a39bf439

How/when did this happen? How was it not noticed?

Change-Id: I352a8dad65f8c27bf5120d567e96d9a12cfa8496

Bug: 8636060
Change-Id: Idfabf7231a80e21786ce1910b5140bb3dddcac7d

This method is confusing with the *Locked convention, and
the two-step call creates a useless callback object. This is
better inlined both for readability and for performance.

Bug: 8636060
Change-Id: I7c427c3ca4e831388a6d54de6728b32206a45d80

The only point of this message is to send the processing on another
thread. However, this will be accomplished later.
Here is the exact call graph:
0. onUpdateBatchInput
1. -> MSG_UPDATE_GESTURE_PREVIEW_AND_SUGGESTION_STRIP
2. -> updateBatchInputSync
3. -> getSuggestedWordsGestureLocked
4. -> MSG_GET_SUGGESTED_WORDS
5. -> LatinIME#getSuggestedWords

The point of both step 1. and step 4. is to make sure the processing
is happening on the InputUpdater thread. Thus, it's useless to do
it twice.

Bug: 11326092
Bug: 8636060
Change-Id: Iceebb9e8879a8f15b73c987f5fd3489f27699be4

Bug: 8636060
And possibly bug: 12085864

Change-Id: I867036bc3d8bd255dc8ceebdd30e28b4b3dc4b2d

This sheds some light on what's happening here. Some
comments were at least misleading, maybe indicating something
is not sequenced as intended.

Bug: 8636060
Change-Id: Ia74feb457a39fe4a672c27fe4203264fda940f04

Change-Id: I503f91e266c71e2370a5807d171e2254c334f7cb

Bug: 8187060
Change-Id: I17be7b77ff89972d3ca6823b087b7bf35105af56

Bug: 12295276
Change-Id: Id359c92acde44758b12929e7bac719d5c9c7577b

Bug: 12295276
Change-Id: I187016b59213d351c943c6e0b9a2ad3708499646

All calls are already passing this

Bug: 8636060
Change-Id: I13accefd7c17039dc5ec12dc9e8133ebc86451ad

Bug: 8636060
Change-Id: I5c47370a3ea5962caff4d40db2faa62accfc7147

No point in calling maybeRetrieveOlderSuggestions now that the
same work is done right here. The callback is also useless now.

Bug: 8636060
Change-Id: Ic14849cb6f6e6293a42cfe068f5c6b18b7677bc0

Bug: 8187060
Change-Id: Ide087f1e39b24695b610bd2b3c464b7e3cb9cd29

Bug: 8187060
Change-Id: I77775aa50763158d99753c2312fa11fe14267aef

I don't see why we would want to remove other suggestions just because
the only suggestion happens to be a dictionary word. That's just
inconsistent.

Bug: 8636060
Change-Id: Ie2d990e5785273d625e6e5518026364165a8f2df

This only gives bad results if a gesture results in only one suggestion.

Change-Id: I5a2f27f83f6901a5a9ecf22f02bdfc9cca6d9f1d

Bug: 8187060
Change-Id: I2f91da12ba2a9e7ce28abd06f314e0e6125d08dc

Move the logic that determines how to handle gesture typing from
PointerTracker to BatchInputArbiter class.

This change must be checked in together with Ia7db8f4fb7.

Change-Id: I5e9f786de58d8a3cb0407ae562af707710ec37fb

Bug: 8636060
Change-Id: I08a6c04986a0bd86ce4c232893e804766ebeebb9

There are far too many one-use methods.

Bug: 8636060
Change-Id: I84e0669b8d2cc5a2817c42b089e874dc60e84769

Bug: 8636060
Change-Id: Id6791085816891c78289b0223bf0807db2da6300