Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 19dab2df authored by Vladislav Kaznacheev's avatar Vladislav Kaznacheev
Browse files

Hide pointer icon while navbar is hidden 

When the navbar gets hidden as a result of
View.setSystemUIVisibility call, the most recent pointer gets
stuck because hover events stop being dispatched to the app.
Hide the pointer icon to avoid confusion.

Bug: 32248070
Test: ApiDemos > View > System UI Visibility > System UI Modes >
   click on HIDE_NAV

Change-Id: Iae80bfa7ab5469946485a68e5308ea752a4645b0
parent 3460fefd

services/core/java/com/android/server/policy/PhoneWindowManager.java

+5 −0
Original line number Diff line number Diff line
@@ -149,6 +149,7 @@ import android.hardware.display.DisplayManager; 
import android.hardware.hdmi.HdmiControlManager;

import android.hardware.hdmi.HdmiPlaybackClient;

import android.hardware.hdmi.HdmiPlaybackClient.OneTouchPlayCallback;

import android.hardware.input.InputManager;

import android.hardware.input.InputManagerInternal;

import android.hardware.power.V1_0.PowerHint;

import android.media.AudioAttributes;
@@ -203,6 +204,7 @@ import android.view.KeyCharacterMap; 
import android.view.KeyCharacterMap.FallbackAction;

import android.view.KeyEvent;

import android.view.MotionEvent;

import android.view.PointerIcon;

import android.view.Surface;

import android.view.View;

import android.view.ViewConfiguration;
@@ -4236,6 +4238,9 @@ public class PhoneWindowManager implements WindowManagerPolicy { 
                mInputConsumer = mWindowManagerFuncs.createInputConsumer(mHandler.getLooper(),

                        INPUT_CONSUMER_NAVIGATION,

                        (channel, looper) -> new HideNavInputEventReceiver(channel, looper));

                // As long as mInputConsumer is active, hover events are not dispatched to the app

                // and the pointer icon is likely to become stale. Hide it to avoid confusion.

                InputManager.getInstance().setPointerIconType(PointerIcon.TYPE_NULL);

            }



            // For purposes of positioning and showing the nav bar, if we have