summaryrefslogtreecommitdiff
path: root/core/java/android/inputmethodservice/InputMethodService.java
diff options
context:
space:
mode:
authorYohei Yukawa <yukawa@google.com>2022-01-11 08:39:58 -0800
committerYohei Yukawa <yukawa@google.com>2022-01-12 02:28:53 +0000
commita25070917a62074d3506b0ded058c3e33390c195 (patch)
treeab7d68672d76c3373f6fa918ba4286e8bd416217 /core/java/android/inputmethodservice/InputMethodService.java
parent5a2fbae44d06d8a821112f7ce042a5160f7a38fc (diff)
Add a sysprop to disable SysUI's nav buttons when IME is shown
This CL introduces a new runtime mode where the SysUI no longer renders the back button and the IME switcher button when the IME is shown, which is the first milestone to move those buttons to the IME process. To enable this mode, run the following commands: $ adb root $ adb shell setprop \ persist.sys.ime.can_render_gestural_nav_buttons true $ adb reboot Keep in mind that you always need to reboot the device after changing this sysprop. Note that only AOSP-based gestural navigation mode is officially supported by this flag, which is when com.android.internal.R.integer.config_navBarInteractionMode is set to WindowManagerPolicyConstants#NAV_BAR_MODE_GESTURAL. Note that nav button rendering in the IME process is yet to be done in subsequent CLs. Hence the expected behavior right now is only one thing that the SysUI stops rendering the back button and the IME switcher button any more when the flag takes effect. The flag is treated as off by default. Hence there should be no visible behavior changes unless you manually enable it. Alternatives Considered: Instead of letting NavigationBarView directly read the sysprop, we could propagate the information from the InputMethodService to the SysUI via InputMethodPrivilegedOperations#setImeWindowStatusAsync(). This approach was abandoned because of the following reasons. * Such a complex mechanism is not necessary for production code. * There are some ad-hoc invocations of IMMS#updateSystemUiLocked(0, mBackDisposition), which makes it difficult to keep IMS and SysUI in sync. Bug: 205803355 Test: Manually done as follows 1. Build aosp_coral-userdebug and flash it 2. Tap any edit field to show AOSP Keyboard. 3. Confirm that nav buttons are rendered in 3-button nav mode 4. Switch to gestural navigation mode. 5. Confirm that nav buttons are still shown when IME is visible. 6. adb root 7. adb shell setprop \ persist.sys.ime.can_render_gestural_nav_buttons true 8. adb reboot 9. Tap any edit field to show AOSP Keyboard. 10. Confirm that nav buttons are not shown when IME is visible. 11. Switch to 3-button navigation mode. 12. Tap any edit field to show AOSP Keyboard. 13. Confirm that nav buttons are shown again when IME is visible. 14. Switch to gestural navigation mode again. 15. Confirm that nav buttons are not shown when IME is visible. Change-Id: I7a2e9f08072a97d61b78dddb144282a5728d1407
Diffstat (limited to 'core/java/android/inputmethodservice/InputMethodService.java')
-rw-r--r--core/java/android/inputmethodservice/InputMethodService.java40
1 files changed, 40 insertions, 0 deletions
diff --git a/core/java/android/inputmethodservice/InputMethodService.java b/core/java/android/inputmethodservice/InputMethodService.java
index afaa085c7cbd..a5005c517535 100644
--- a/core/java/android/inputmethodservice/InputMethodService.java
+++ b/core/java/android/inputmethodservice/InputMethodService.java
@@ -54,6 +54,7 @@ import static android.view.WindowInsets.Type.statusBars;
import static java.lang.annotation.RetentionPolicy.SOURCE;
+import android.annotation.AnyThread;
import android.annotation.CallSuper;
import android.annotation.DrawableRes;
import android.annotation.IntDef;
@@ -83,6 +84,7 @@ import android.os.Handler;
import android.os.IBinder;
import android.os.ResultReceiver;
import android.os.SystemClock;
+import android.os.SystemProperties;
import android.os.Trace;
import android.provider.Settings;
import android.text.InputType;
@@ -302,6 +304,44 @@ public class InputMethodService extends AbstractInputMethodService {
static final boolean DEBUG = false;
/**
+ * Key for a boolean value that tells whether {@link InputMethodService} is responsible for
+ * rendering the back button and the IME switcher button or not when the gestural navigation is
+ * enabled.
+ *
+ * <p>This sysprop is just ignored when the gestural navigation mode is not enabled.</p>
+ *
+ * <p>
+ * To avoid complexity that is not necessary for production, you always need to reboot the
+ * device after modifying this flag as follows:
+ * <pre>
+ * $ adb root
+ * $ adb shell setprop persist.sys.ime.can_render_gestural_nav_buttons true
+ * $ adb reboot
+ * </pre>
+ * </p>
+ */
+ private static final String PROP_CAN_RENDER_GESTURAL_NAV_BUTTONS =
+ "persist.sys.ime.can_render_gestural_nav_buttons";
+
+ /**
+ * Returns whether {@link InputMethodService} is responsible for rendering the back button and
+ * the IME switcher button or not when the gestural navigation is enabled.
+ *
+ * <p>This method is supposed to be used with an assumption that the same value is returned in
+ * other processes. It is developers' responsibility for rebooting the device when the sysprop
+ * is modified.</p>
+ *
+ * @return {@code true} if {@link InputMethodService} is responsible for rendering the back
+ * button and the IME switcher button when the gestural navigation is enabled.
+ *
+ * @hide
+ */
+ @AnyThread
+ public static boolean canImeRenderGesturalNavButtons() {
+ return SystemProperties.getBoolean(PROP_CAN_RENDER_GESTURAL_NAV_BUTTONS, false);
+ }
+
+ /**
* Allows the system to optimize the back button affordance based on the presence of software
* keyboard.
*