Bug: 8636060
Change-Id: I5d08ed5d69cdb03bd188e36b7014de03f04fa72f

Change-Id: I96fc381d2b45881aeb342d33207cd7fe82f5077c

Bug: 11757851

Change-Id: I94f32859d862fe41e39845ab2bfc2d396c6aea0f

Bug: 8636060
Change-Id: I0480ac03b9902e940f566b5558b8499647983968

Change-Id: I8f5d8fd8c1627a568df57255dc3dceda26c4720c

Bug: 8636060
Change-Id: If0309be038518a71d735108b78ac9efbfe38c82b

Change-Id: I8a8aa9f703c6fca5e25ec573b41f325680ac92dc

Bug: 8636060
Change-Id: I45b322e4844f023e91b80f9c2e0f7fa0166bcc59

I don't understand why this was checked in. The tests are completely
useless, that makes the variables write-only, and the only side effect
is useless as the keyboard commits suicide upon exiting debug
settings.

Change-Id: I4eb25295183f43a2e947a6a0467ea168d023c18a

Bug: 8636060
Change-Id: Iedfdf71350f5fe3ab451fc5ec6e3c9901f71e50f

Bug: 8636060
Change-Id: I373874585f4fa663b4207b9a02d751805259eb36

Only ever used here.

Bug: 8636060
Change-Id: Ia98c8bbe1c63ad4d057efe1cee87829b06124479

Bug: 8636060
Change-Id: Id57ced0cca9dc9e5e85c3a53683588feb28c1402

Change-Id: I6307c1545d9d48cdea66d08f8efc1b7fbb08ee42

Change-Id: I6b957b43c6dd7ce4f639e46d248179662c1ee669

Bug: 8636060
Change-Id: I3a840c30cc4489debf9118beeec448488503bc1c

Also pull the TLD specific processing

Bug: 8636060
Change-Id: I20931ac6fc6c55a05aa8f8b97b0210c07d230ea2

Bug: 8636060
Change-Id: I566f9e109ebcabe3a6a778118434ce79d02d1113

If we have the same text field, then the target app is
necessarily the same. No need to re-evaluate it.

Bug: 8636060
Change-Id: I66eb194c4cd08a6c1d47b26b99bd7f420130097e

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

Dynamically test for the presence of main dict suggestions. This
is much more potent and more accurate than the vaguely defined
boolean.
Also, resolve a TODO and avoid creating an object uselessly.

Not directly related, but helpful with:
Bug: 8636060

Change-Id: Ib1745f77ee6d9ec7cd8bbfa5a548652ec84ec902

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

Bug: 8636060
Change-Id: I8c5c028d81183adb0167b2034ee01de8048edafb

Create an InputEngine class and start moving the relevant
variables to it.

Bug: 8636060
Change-Id: I64adef4ac01e665c19bb7c5b93517a0cbd03d817

Bug: 11142685
Change-Id: Ic3a5d452f6c325248ac4e1bfa37774c58586f359

Bug: 11757851
Change-Id: I83e484195f036e35cbae21fe63148bc9c22bfad7