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

Bug: 12390573
Change-Id: I6f04f07a46babbc3d51cb94c7eaee21bed242396

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

Bug: 8636060
Change-Id: Idfabf7231a80e21786ce1910b5140bb3dddcac7d

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

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

Bug: 8187060
Change-Id: I77775aa50763158d99753c2312fa11fe14267aef

Bug: 8636060
Change-Id: Id6791085816891c78289b0223bf0807db2da6300

Bug: 8187060

Change-Id: Iabe13abcd43b381360f7080b3f4ee0533f246160

Bug: 8636060
Change-Id: I5d08ed5d69cdb03bd188e36b7014de03f04fa72f

Bug: 8636060
Change-Id: I0480ac03b9902e940f566b5558b8499647983968

Change-Id: I8f5d8fd8c1627a568df57255dc3dceda26c4720c

Bug: 8636060
Change-Id: If0309be038518a71d735108b78ac9efbfe38c82b

Bug: 8636060
Change-Id: I45b322e4844f023e91b80f9c2e0f7fa0166bcc59

Bug: 8636060
Change-Id: I373874585f4fa663b4207b9a02d751805259eb36

Bug: 8636060
Change-Id: Id57ced0cca9dc9e5e85c3a53683588feb28c1402

Bug: 8636060
Change-Id: I3a840c30cc4489debf9118beeec448488503bc1c

Also pull the TLD specific processing

Bug: 8636060
Change-Id: I20931ac6fc6c55a05aa8f8b97b0210c07d230ea2

Bug: 8636060
Change-Id: I566f9e109ebcabe3a6a778118434ce79d02d1113

Bug: 8636060
Change-Id: Iec7c1001c46cf8e9a44422e3f9008e437cc5cab6

Bug: 8636060
Change-Id: I3d00095cd0e83d1ee3834b37ff188b5aa9806fcb

Also not-cache a value whose documentation says not to cache
it. That makes it write only, so remove it.

Bug: 8636060
Change-Id: I39798a039cebf79ad78a41fe457c9d9764a3f275

Bug: 8636060
Change-Id: I4bfdf1628c9b55a79bd2d88ee30bf10b08a49232

Bug: 8636060
Change-Id: I6338dcee0f7a80f2181ba0369f71350cb21d231a

It's only ever used in one place and makes unhealthy assumptions
about what the caller is doing. It's better inlined.

Bug: 8636060
Change-Id: I265d5d96894e929d09247ee2e3dc2e81efc2a276

Bug: 8636060
Change-Id: I99474248e8fa35eb3053b5520abc8716805fdfb4

Make some methods private and add comments.

Bug: 8636060
Change-Id: I6c7d13dca8a22dd1a6f110e8b18d52864429579f

Bug: 8636060
Change-Id: Ia5788b365f05ae880bad957cbef00ecc93abb5d1

Bug: 8636060
Change-Id: I405c4a537858a6d5ab0b29502a2792e8c3b2564e

Also move getActualCapsMode and getNthPreviousWordForSuggestion

Bug: 8636060
Change-Id: I1ee9162d0b7a517070c4b7420b084c973f061533

Bug: 8636060
Change-Id: I5bf2b1fa2f9338fe28151e66eac1ceda6dbd4a7f

Also move swapSwapperAndSpace, maybeDoubleSpacePeriod,
maybeStripSpace, promotePhantomSpace and canBeFollowedByDoubleSpacePeriod

Bug: 8636060
Change-Id: I864053d21c4b3d51ee7a3cd79053ca602337f6f8

Bug: 8636060
Change-Id: Id76889f8405458bcacb9c7c694cb193b27b842f3

Bug: 8636060
Change-Id: I9c6869c85db49d7f87e95a421389eaf47ae3ab2e

Also move resetEntireInputState, resetComposingState,
sendDownUpKeyEvent, sendKeyCodePoint and getTextWithUnderline

Bug: 8636060
Change-Id: Ic4c270ac49769ffba41dc3f12d16ac3aa938717f