Bug: 6294817
Change-Id: I5010eafa5ba7e947743706adf7e722f4f0cfb415

Bug: 6279966
Change-Id: Ia94a78d79cc51014126d18612aabdc7ce51487a4

Bug: 5240798
Change-Id: Ie56c1c2cfd7f365e771fee88c1ed15012448feed

Change-Id: I6d91f3b086470b79306dbe2874db9748b9e0eb5f

I tracked all the calls to the constructor, and the passed
list is never null. It's also final, so it's safe.

Change-Id: I320f907acee0b237bb286a9a183c18106809174a

All calls to this methods are made with an actual String. No sense
beating ourselves with an interface stick.

Change-Id: I6ef98286be6f81f73864b04f3a17e68f36a6e542

Change-Id: I6950897d262f610d502b7f8293b9dd8d2b53e6e8

This has been moved to the SuggestedWords object and is now
represented by a single flag instead of a flag in each suggestion.

Change-Id: I6e9a7cff5a7701de7e61b4de13baac25c9a87b1b

Change-Id: I9684c7b08244b34853ce8a99b6e9d885389f6687

Change-Id: Ide3f605ce40e8c230f217bbf37ab016457ee91f1

Change-Id: I03005240238fb004c20b2df0d836796e2e8b307f

Change-Id: I208dd2e725e37c2fb3cd51fe48194679e5912f6b

Change-Id: Idef22e6a82d59e3ec95b95ad237debcb07daf000

We have to match one way or another, and the argument order to the
Builder call is more logical.

Change-Id: Iac7c3a351c2687cb294d6a4924fd9cb20ca95177

Change-Id: I2184084f4b7729c4324f8291d548bd4c4c2872a1

Change-Id: Ic3e07b745b7b24cae0f772ea49e0de615b44caf9

I had missed that one

Change-Id: I9a5040a7a61e294396aa424de96a69dbb15f037a

Change-Id: Ie399ca7a9e76ccab44a92bc378d11f92392fed2c

...and go for the kill

Change-Id: I5ad57211b250d2578ca22a5dae93344d52f9da80

Also stop using Builder between its creation and the call to
the #build() method.

Change-Id: Ie1fc3ec7b6f4c7c3789f672f4e26b4bf58c3e062

Change-Id: Ic455f97247da5e40d4939555ab639ecffef36e2d

Use the same local variable inside both branches of an if.

Change-Id: I61f7d506d984f3723ec90604416d1875dd97cd8c

Change-Id: I3132b7fe4ee97c566736bfc21c3783cfc79fc7ff

Change-Id: I2ddfab00dc8c141fc989a051f16eb2b3571b7e86

Remove 2 methods that were left only for tests to compile in
another project, but are not used any more now that the tests
have been updated.
Also remove an attribute that is never read.

Change-Id: Ic9ba1f52a465199c82c923d77ed7450460e8a4b9

This method now only sets words, so it should be named set.
The functionality is identical since there are no more places
where the list is reused.
This will also allow to make the list final in an upcoming change.

Change-Id: I25b0c7d7f13c3fa5d89806f01f48f1026769603f

Remove an unused method, make others static, fix a formatting
issue

Change-Id: I40cb0b5e01a6f8e2f7b953bd9aad3168dfee71d1

...and replace by a call to a central method.

Change-Id: I93d0a2c2e99963a5b69923d1062d0e01853216b6

Change-Id: I7625bf17158c207ad7e8cd496bef825704173c4c

...working this time

Change-Id: I80e377e6250d3817f1e067a551bca2a557740764

This reverts commit ca0e0486

Change-Id: I42baeec68eb2540e7d8f8538ae2dceeef5594391

We can't get rid of SuggestedWordInfo because it conveys
the information about whether a suggestion is outdated or
not. It is always created anyway. So it makes sense not
to have separate lists and to just keep everything in the
SuggestedWordInfo object.

Change-Id: Idd79dc267d97633901d3d2869da457736963e178

Tests are using a wrapper method that's going to disappear soon.
The method that becomes package-visible now is destined to be
removed too a little later.

Change-Id: If99b1c77ed52e050b47216cc7afab5f35c4bbb38

Change-Id: Ie1d460562d38105abe91ae2da17cafd49e2cd394

mHasAutoCorrectionCandidate now includes the fact that this
did not get caught by the safety net. This is more intuitive.

Change-Id: I6502bbf1ca64a0a6d64f55d723305cf78352a656

Change-Id: I10959d47c980d2e8f2ea995955acf02193e3ff7b

Change-Id: I1ef0685cc9888c9cef9f60015d32b66dde75ef9b

Change-Id: Ib264533a05e9e09347bf254789e6ab5beec92400

Hopefully that will end in Suggest/SuggestedWords being autonomous
and won't need the logic spoon-fed to them

Change-Id: I915661bce13c69c8a5b8e5d4a8c41e18fea594cf