See https://source.android.com/setup/contribute/respectful-code for reference

Bug: 161896447
Test: tapas LatinIME && m
Change-Id: I2ffb39ff70f61e579b62761d3da526a545dead1f

Bug: 20641948
Change-Id: I6639c995b12c033bc30241cd219201dd483ee516

We only have "en" included in the APK. Everything else shows a stack trace.

Bug 19946159

Change-Id: Ie213529d52175eb929b2c875a5558aea347af01f

Bug:19961803
Change-Id: I464e840bc5f75d164a519d84b0f09d8e64ed5061

Needed for unit tests related to various bug fixes.

Bug 19930761.

Change-Id: I776ccccb032e3d1b181b02c6bb768500790870f7

Change-Id: I0ec4aa69d9b5f013ae926cc368e25225d9d3073b

Change-Id: I666308ada968ee7993235ad98b92f2ad522af122

Also added toString for AssetFileAddress for easier debugging.

Change-Id: Ie5d15d12bae46c31b28d8c67002e627f4c6690e3

Also why did we have two copies of LocaleUtils >.>

Bug: 18108776
Change-Id: I03b4403dfd51934e66b567f2f8b87da419cfb3ab

This CL also adds @SuppressWarning("unused" to java-overridable package.

Bug: 18003991
Change-Id: If70527e30654384705d7a814f5efd181d9f539e1

This CL fixes the following compiler warnings.

- Indirect access to static member
- Access to a non-accessible member of an enclosing type
- Parameter assignment
- Method can be static
- Local variable declaration hides another field or variable
- Value of local variable is not used
- Unused import
- Unused private member
- Unnecessary 'else' statement
- Unnecessary declaration of throw exception
- Redundant type arguments
- Missing '@Override' annotation
- Unused '@SuppressWarning' annotations

Bug: 18003991
Change-Id: Icfebe753e53a2cc621848f769d6a3d7ce501ebc7

Change-Id: If16ef50ae73147594615d0f49d6a22621eaf1aef

Bug: 8187060
Bug:13035567
Change-Id: I90a5e0665c367f5a1bd786f4babf0d03e79e68fa

Bug: 13164518
Change-Id: I8768ad887af8b89ad9f29637f606c3c68629c7ca

Bug: 12810574
Change-Id: Iacf1cd05a268bf690ab864b5e32a18a4b0ccc693

Change-Id: I5cfa0d2fccc139bd6c45c5590a68c3e0c90534b8

Apparently, it is possible that File#getPath() returns null.
In this case, we'll have a null AssetFileAddress and an NPE
later down the road.

Bug: 9944824
Change-Id: Id6ceac1cb55439660f9811cf0fe3b574200d5a20

Bug: 9618601
Change-Id: I173100ac704c03f7d5d0d53477e83cab5d1110d4

Bug: 10434720
Change-Id: I14690a6e0f922ed1bab3a4b6c9a457ae84d4c1a4

Change-Id: Ibf9b95b658df6e2c2218bdb62e2380f326a03832

Change-Id: I1a1830aaa8ea586b68fc34ff3a27ae52b810e8af

BinaryDictReader -> BinaryDictDecoder.
BinaryDictDecoder -> BianryDictDecoderUtils.

Change-Id: Iadf2153b379b760538ecda488dda4f17225e5f37

Change-Id: I7c3269d77e3e3b567e459dcaa1bc029903941744

Revert "[Refactor] Divide BinaryDictInputOutput into BinaryDictInputUtils and BinaryDictOutputUtils."

This reverts commit 4c63d061.

Change-Id: I1fa277d720bab4d895259df7d6d82eebfa5eb6c5

Change-Id: I0d476abe763c11ba9005152f928e8dccf15ac9de

Change-Id: I827052f234eeaa4dbcfd37da69a99866896a158b

Change-Id: I7290cd1fb675a1b85b9b6ac2d464c932b5bca1dd

Bug: 9617181
Change-Id: I228a3eac959f81b38e4092015e8e3c429d46fa60

There is no reason not to contact the dictionary provider
when we don't have internet permission or when the URL
is empty. It knows how to handle both these cases.

Bug: 9388602
Change-Id: I30c4540551ad2f5e527d3acd1842bbd749feca89

Mostly remove unused imports

Change-Id: If8ceffdfad79b78194094527759f5c334746973e

Change-Id: I1c5b27c8edf231680edb8d96f63b9d04cfc6a6fa

Bug: 9305379

Change-Id: Iab32753070a056d3861a45aa898911b7ab67dfb6

Change-Id: I974964aa3cc24b5480e57bd38c197fe7a49acaa4

Change-Id: Iac388c8aefc684980cb4c7131fb47319f6073eb3

Bug: 8715614
Change-Id: I307d83923070994525cf0d27aad403e881a89314

Bug: 8161354
Change-Id: I17c23f56dd3bc2f27726556bf2c5a9d5520bd172

bug: 8393568
Change-Id: Ie5edf44a3627aca9416145aff56bf05bbf2a05f3