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

Commit 8705444e authored by Sandro Meier's avatar Sandro Meier Committed by Vladimir Komsiyski
Browse files

Inline getVirtualDisplayName into deprecated API 

The getVirtualDisplayName is only used in the deprecated virtual display
creation API. The new API uses VirtualDisplayConfig where the client can
set the name.
To make it clear the name as generated is only used in the deprecated
API this commit removes the private method and inlines it in the
deprecated constructor.

Bug: 270334704
Test: VDM Test app, created Virtual display with old and new API
Change-Id: I73726b73aef94276d44a830f83ad74fc764bc1fb
(cherry picked from commit d744bd18)
parent 9dfeef8c

core/java/android/companion/virtual/VirtualDeviceManager.java

+6 −13
Original line number Diff line number Diff line
@@ -540,8 +540,13 @@ public final class VirtualDeviceManager { 
                @VirtualDisplayFlag int flags,

                @Nullable @CallbackExecutor Executor executor,

                @Nullable VirtualDisplay.Callback callback) {

            // Currently this just use the device ID, which means all of the virtual displays

            // created using the same virtual device will have the same name if they use this

            // deprecated API. The name should only be used for informational purposes, and not for

            // identifying the display in code.

            String virtualDisplayName =  "VirtualDevice_" + getDeviceId();

            VirtualDisplayConfig.Builder builder = new VirtualDisplayConfig.Builder(

                    getVirtualDisplayName(), width, height, densityDpi)

                    virtualDisplayName, width, height, densityDpi)

                    .setFlags(flags);

            if (surface != null) {

                builder.setSurface(surface);
@@ -825,18 +830,6 @@ public final class VirtualDeviceManager { 
            }

        }



        private String getVirtualDisplayName() {

            try {

                // Currently this just use the device ID, which means all of the virtual displays

                // created using the same virtual device will have the same name. The name should

                // only be used for informational purposes, and not for identifying the display in

                // code.

                return "VirtualDevice_" + mVirtualDevice.getDeviceId();

            } catch (RemoteException e) {

                throw e.rethrowFromSystemServer();

            }

        }



        /**

         * Adds an activity listener to listen for events such as top activity change or virtual

         * display task stack became empty.