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

Commit 8e73ebe7 authored by Greg Kaiser's avatar Greg Kaiser Committed by Android (Google) Code Review
Browse files

Revert "[CDM][Refactoring 4/N] Move device presence logic out of..." 

Revert submission 26624172-cdm-cleanup4

Reason for revert: Main suspect for b/330812093

Reverted changes: /q/submissionid:26624172-cdm-cleanup4

Change-Id: I04267bf9762e4e34763019528425f32d95f2fd41
parent a28469fe

core/java/android/companion/CompanionDeviceManager.java

+4 −4
Original line number Diff line number Diff line
@@ -1086,7 +1086,7 @@ public final class CompanionDeviceManager { 
        }

        Objects.requireNonNull(deviceAddress, "address cannot be null");

        try {

            mService.legacyStartObservingDevicePresence(deviceAddress,

            mService.registerDevicePresenceListenerService(deviceAddress,

                    mContext.getOpPackageName(), mContext.getUserId());

        } catch (RemoteException e) {

            ExceptionUtils.propagateIfInstanceOf(e.getCause(), DeviceNotAssociatedException.class);
@@ -1128,7 +1128,7 @@ public final class CompanionDeviceManager { 
        }

        Objects.requireNonNull(deviceAddress, "address cannot be null");

        try {

            mService.legacyStopObservingDevicePresence(deviceAddress,

            mService.unregisterDevicePresenceListenerService(deviceAddress,

                    mContext.getPackageName(), mContext.getUserId());

        } catch (RemoteException e) {

            ExceptionUtils.propagateIfInstanceOf(e.getCause(), DeviceNotAssociatedException.class);
@@ -1328,7 +1328,7 @@ public final class CompanionDeviceManager { 
    @RequiresPermission(android.Manifest.permission.REQUEST_COMPANION_SELF_MANAGED)

    public void notifyDeviceAppeared(int associationId) {

        try {

            mService.notifySelfManagedDeviceAppeared(associationId);

            mService.notifyDeviceAppeared(associationId);

        } catch (RemoteException e) {

            throw e.rethrowFromSystemServer();

        }
@@ -1350,7 +1350,7 @@ public final class CompanionDeviceManager { 
    @RequiresPermission(android.Manifest.permission.REQUEST_COMPANION_SELF_MANAGED)

    public void notifyDeviceDisappeared(int associationId) {

        try {

            mService.notifySelfManagedDeviceDisappeared(associationId);

            mService.notifyDeviceDisappeared(associationId);

        } catch (RemoteException e) {

            throw e.rethrowFromSystemServer();

        }

core/java/android/companion/ICompanionDeviceManager.aidl

+12 −12
Original line number Diff line number Diff line
@@ -59,16 +59,12 @@ interface ICompanionDeviceManager { 
        int userId);



    @EnforcePermission("REQUEST_OBSERVE_COMPANION_DEVICE_PRESENCE")

    void legacyStartObservingDevicePresence(in String deviceAddress, in String callingPackage, int userId);



    @EnforcePermission("REQUEST_OBSERVE_COMPANION_DEVICE_PRESENCE")

    void legacyStopObservingDevicePresence(in String deviceAddress, in String callingPackage, int userId);



    @EnforcePermission("REQUEST_OBSERVE_COMPANION_DEVICE_PRESENCE")

    void startObservingDevicePresence(in ObservingDevicePresenceRequest request, in String packageName, int userId);

    void registerDevicePresenceListenerService(in String deviceAddress, in String callingPackage,

        int userId);



    @EnforcePermission("REQUEST_OBSERVE_COMPANION_DEVICE_PRESENCE")

    void stopObservingDevicePresence(in ObservingDevicePresenceRequest request, in String packageName, int userId);

    void unregisterDevicePresenceListenerService(in String deviceAddress, in String callingPackage,

        int userId);



    boolean canPairWithoutPrompt(in String packageName, in String deviceMacAddress, int userId);
@@ -97,11 +93,9 @@ interface ICompanionDeviceManager { 
    @EnforcePermission("USE_COMPANION_TRANSPORTS")

    void removeOnMessageReceivedListener(int messageType, IOnMessageReceivedListener listener);



    @EnforcePermission("REQUEST_COMPANION_SELF_MANAGED")

    void notifySelfManagedDeviceAppeared(int associationId);

    void notifyDeviceAppeared(int associationId);



    @EnforcePermission("REQUEST_COMPANION_SELF_MANAGED")

    void notifySelfManagedDeviceDisappeared(int associationId);

    void notifyDeviceDisappeared(int associationId);



    PendingIntent buildPermissionTransferUserConsentIntent(String callingPackage, int userId,

        int associationId);
@@ -141,4 +135,10 @@ interface ICompanionDeviceManager { 
    byte[] getBackupPayload(int userId);



    void applyRestoredPayload(in byte[] payload, int userId);



    @EnforcePermission("REQUEST_OBSERVE_COMPANION_DEVICE_PRESENCE")

    void startObservingDevicePresence(in ObservingDevicePresenceRequest request, in String packageName, int userId);



    @EnforcePermission("REQUEST_OBSERVE_COMPANION_DEVICE_PRESENCE")

    void stopObservingDevicePresence(in ObservingDevicePresenceRequest request, in String packageName, int userId);

}

services/companion/java/com/android/server/companion/presence/CompanionAppBinder.java→services/companion/java/com/android/server/companion/CompanionApplicationController.java

+567 −0

File changed and moved.

Preview size limit exceeded, changes collapsed.

services/companion/java/com/android/server/companion/CompanionDeviceManagerService.java

+492 −57

File changed.

Preview size limit exceeded, changes collapsed.

services/companion/java/com/android/server/companion/CompanionDeviceManagerServiceInternal.java

+5 −0
Original line number Diff line number Diff line
@@ -27,6 +27,11 @@ import java.util.Collection; 
 * Companion Device Manager Local System Service Interface.

 */

public interface CompanionDeviceManagerServiceInternal {

    /**

     * @see CompanionDeviceManagerService#removeInactiveSelfManagedAssociations

     */

    void removeInactiveSelfManagedAssociations();



    /**

     * Registers a callback from an InCallService / ConnectionService to CDM to process sync

     * requests and perform call control actions.