diff options
Diffstat (limited to 'core/java/android')
| -rw-r--r-- | core/java/android/companion/IOnAssociationsChangedListener.aidl | 19 |
1 files changed, 18 insertions, 1 deletions
diff --git a/core/java/android/companion/IOnAssociationsChangedListener.aidl b/core/java/android/companion/IOnAssociationsChangedListener.aidl index e6794b741e46..d3694564ab7b 100644 --- a/core/java/android/companion/IOnAssociationsChangedListener.aidl +++ b/core/java/android/companion/IOnAssociationsChangedListener.aidl @@ -20,5 +20,22 @@ import android.companion.AssociationInfo; /** @hide */ interface IOnAssociationsChangedListener { - oneway void onAssociationsChanged(in List<AssociationInfo> associations); + + /* + * IMPORTANT: This method is intentionally NOT "oneway". + * + * The method is intentionally "blocking" to make sure that the clients of the + * addOnAssociationsChangedListener() API (@SystemAPI guarded by a "signature" permission) are + * able to prevent race conditions that may arise if their own clients (applications) + * effectively get notified about the changes before system services do. + * + * This is safe for 2 reasons: + * 1. The addOnAssociationsChangedListener() is only available to the system components + * (guarded by a "signature" permission). + * See android.permission.MANAGE_COMPANION_DEVICES. + * 2. On the Java side addOnAssociationsChangedListener() in CDM takes an Executor, and the + * proxy implementation of onAssociationsChanged() simply "post" a Runnable to it. + * See CompanionDeviceManager.OnAssociationsChangedListenerProxy class. + */ + void onAssociationsChanged(in List<AssociationInfo> associations); }
\ No newline at end of file |