OSDN Git Service

Deprecate IMM#showSoftInputUnchecked() part 2
authorYohei Yukawa <yukawa@google.com>
Tue, 21 Mar 2017 19:38:44 +0000 (12:38 -0700)
committerYohei Yukawa <yukawa@google.com>
Tue, 21 Mar 2017 19:38:44 +0000 (12:38 -0700)
commitf6af093e2dfd156768a3937e94d405c04c541fea
tree1a961d69441fd98b71d9b918004a37f13caebc3d
parente39e8350a12acabe19dcf8aae7cc9064665ce71c
Deprecate IMM#showSoftInputUnchecked() part 2

This is the 2nd attempt to have CryptKeeper stop depending on
InputMethodManager#showSoftInputUnchecked() do avoid non-deterministic
behaviors.  Note that the original one [1] was reverted [2] due to
ClassCastException as crypt_keeper_pin_field.xml was not updated to
use ImeAwareEditText.

We have been receiving more and more strange bug reports regarding
the initial IME visibility on CryptKeeper (Bug 31915865, Bug 35198715,
and Bug 36148078).  Most likely we have introduced more bugs before
fixing previous bugs.

As the first step to disentangle multiple problems, this CL addresses
a long standing issue that CryptKeeper relies on @hide internal API
InputMethodManager#showSoftInputUnchecked().  As explained in
Bug 36015425, IMM#showSoftInputUnchecked() is considered harmful in
terms of bypassing standard IME focus handling flows, which prevents
us from taking advantage of historical debugging mechanism that we
have added recently in Bug 35079353.

 [1]: I3cffeec286d9372ff05e85a49b82287cb1add8a2
      4ae46cd1ad0725c8049c3954de19874df8483983
 [2]: I776ad2e6ba0eabacb3d6be6de23ba37fe5ce7bc7
      cf026ead8009578e91cc2daf3ff69aed4c4c9261

Test: Manually verified that Bug 31915865 is still broken but this CL
      does not make it worse.
      1. Flash an image into a non direct-boot device.
      2. Set up a device boot password.
      3. adb reboot
      After the step 3, the software keyboard shows up automatically
      then suddenly disappears (Bug 35198715). Do the same test for
      device boot PIN instead of password.
Bug: 36015425
Change-Id: I182b05d3ff59fb3b4732d60d0d5a464f0e0e0235
res/layout/crypt_keeper_password_field.xml
res/layout/crypt_keeper_pin_field.xml
src/com/android/settings/CryptKeeper.java
src/com/android/settings/widget/ImeAwareEditText.java [new file with mode: 0644]