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

Commit e244db42 authored by Jesse Hall's avatar Jesse Hall
Browse files

Handle hotplug events as described instead of rescanning 

Hotplug events say which display they're for and whether the display
was connected or disconnected. Before, this info was ignored, and the
event just triggered a rescan of all displays. If a display was
disconnected and then reconnected quickly, the rescan would treat this
as a no-op or a device property change and wouldn't turn the display
on.

Now the display manager attempts to update its state with the change
the event describes. So a quick disconnect/connect cycle will cause
the display to be turned on since the display manager will have
updated its internal state to reflect the disconnect event, and will
treat the connect event as a new display rather than a device property
change.

Bug: 7491120
Change-Id: If16268fbd0708683f6401ea72fbee9fb2c3c8d19
parent 5756c300

services/java/com/android/server/display/LocalDisplayAdapter.java

+34 −23
Original line number Diff line number Diff line
@@ -60,11 +60,13 @@ final class LocalDisplayAdapter extends DisplayAdapter { 
        super.registerLocked();



        mHotplugReceiver = new HotplugDisplayEventReceiver(getHandler().getLooper());

        scanDisplaysLocked();

    }



    private void scanDisplaysLocked() {

        for (int builtInDisplayId : BUILT_IN_DISPLAY_IDS_TO_SCAN) {

            tryConnectDisplayLocked(builtInDisplayId);

        }

    }



    private void tryConnectDisplayLocked(int builtInDisplayId) {

        IBinder displayToken = Surface.getBuiltInDisplay(builtInDisplayId);

        if (displayToken != null && Surface.getDisplayInfo(displayToken, mTempPhys)) {

            LocalDisplayDevice device = mDevices.get(builtInDisplayId);
@@ -78,6 +80,13 @@ final class LocalDisplayAdapter extends DisplayAdapter { 
                sendDisplayDeviceEventLocked(device, DISPLAY_DEVICE_EVENT_CHANGED);

            }

        } else {

            // The display is no longer available. Ignore the attempt to add it.

            // If it was connected but has already been disconnected, we'll get a

            // disconnect event that will remove it from mDevices.

        }

    }



    private void tryDisconnectDisplayLocked(int builtInDisplayId) {

        LocalDisplayDevice device = mDevices.get(builtInDisplayId);

        if (device != null) {

            // Display was removed.
@@ -85,8 +94,6 @@ final class LocalDisplayAdapter extends DisplayAdapter { 
            sendDisplayDeviceEventLocked(device, DISPLAY_DEVICE_EVENT_REMOVED);

        }

    }

        }

    }



    private final class LocalDisplayDevice extends DisplayDevice {

        private final int mBuiltInDisplayId;
@@ -191,7 +198,11 @@ final class LocalDisplayAdapter extends DisplayAdapter { 
        @Override

        public void onHotplug(long timestampNanos, int builtInDisplayId, boolean connected) {

            synchronized (getSyncRoot()) {

                scanDisplaysLocked();

                if (connected) {

                    tryConnectDisplayLocked(builtInDisplayId);

                } else {

                    tryDisconnectDisplayLocked(builtInDisplayId);

                }

            }

        }

    }