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

Commit 96d0c530 authored by Arthur Ishiguro's avatar Arthur Ishiguro
Browse files

Adds missing permission annotation to HubEndpointSession

Also adds other missing annotation in .aidl file.

Bug: 381090691
Flag: android.chre.flags.offload_api
Test: Compile
Change-Id: I6dd4a961ecf8d4d7bf3c4b9ef0ac6e2883caad39
parent c0bf94ea
Loading
Loading
Loading
Loading
+2 −2
Original line number Diff line number Diff line
@@ -5252,9 +5252,9 @@ package android.hardware.contexthub {
  }
  @FlaggedApi("android.chre.flags.offload_api") public class HubEndpointSession implements java.lang.AutoCloseable {
    method public void close();
    method @RequiresPermission(android.Manifest.permission.ACCESS_CONTEXT_HUB) public void close();
    method @Nullable public android.hardware.contexthub.HubServiceInfo getServiceInfo();
    method @NonNull public android.hardware.location.ContextHubTransaction<java.lang.Void> sendMessage(@NonNull android.hardware.contexthub.HubMessage);
    method @NonNull @RequiresPermission(android.Manifest.permission.ACCESS_CONTEXT_HUB) public android.hardware.location.ContextHubTransaction<java.lang.Void> sendMessage(@NonNull android.hardware.contexthub.HubMessage);
  }
  @FlaggedApi("android.chre.flags.offload_api") public class HubEndpointSessionResult {
+3 −0
Original line number Diff line number Diff line
@@ -19,6 +19,7 @@ package android.hardware.contexthub;
import android.annotation.FlaggedApi;
import android.annotation.NonNull;
import android.annotation.Nullable;
import android.annotation.RequiresPermission;
import android.annotation.SystemApi;
import android.chre.flags.Flags;
import android.hardware.location.ContextHubTransaction;
@@ -70,6 +71,7 @@ public class HubEndpointSession implements AutoCloseable {
     *     receiving the response for the message.
     */
    @NonNull
    @RequiresPermission(android.Manifest.permission.ACCESS_CONTEXT_HUB)
    public ContextHubTransaction<Void> sendMessage(@NonNull HubMessage message) {
        if (mIsClosed.get()) {
            throw new IllegalStateException("Session is already closed.");
@@ -120,6 +122,7 @@ public class HubEndpointSession implements AutoCloseable {
     * <p>When this function is invoked, the messaging associated with this session is invalidated.
     * All futures messages targeted for this client are dropped.
     */
    @RequiresPermission(android.Manifest.permission.ACCESS_CONTEXT_HUB)
    public void close() {
        if (!mIsClosed.getAndSet(true)) {
            mCloseGuard.close();
+6 −0
Original line number Diff line number Diff line
@@ -38,6 +38,7 @@ interface IContextHubEndpoint {
     * @throws IllegalArgumentException If the HubEndpointInfo is not valid.
     * @throws IllegalStateException If there are too many opened sessions.
     */
    @EnforcePermission("ACCESS_CONTEXT_HUB")
    int openSession(in HubEndpointInfo destination, in @nullable HubServiceInfo serviceInfo);

    /**
@@ -48,6 +49,7 @@ interface IContextHubEndpoint {
     *
     * @throws IllegalStateException If the session wasn't opened.
     */
    @EnforcePermission("ACCESS_CONTEXT_HUB")
    void closeSession(int sessionId, int reason);

    /**
@@ -59,11 +61,13 @@ interface IContextHubEndpoint {
     *
     * @throws IllegalStateException If the session wasn't opened.
     */
    @EnforcePermission("ACCESS_CONTEXT_HUB")
    void openSessionRequestComplete(int sessionId);

    /**
     * Unregister this endpoint from the HAL, invalidate the EndpointInfo previously assigned.
     */
    @EnforcePermission("ACCESS_CONTEXT_HUB")
    void unregister();

    /**
@@ -75,6 +79,7 @@ interface IContextHubEndpoint {
     * @param transactionCallback Nullable. If the hub message requires a reply, the transactionCallback
     *                            will be set to non-null.
     */
    @EnforcePermission("ACCESS_CONTEXT_HUB")
    void sendMessage(int sessionId, in HubMessage message,
                     in @nullable IContextHubTransactionCallback transactionCallback);

@@ -86,5 +91,6 @@ interface IContextHubEndpoint {
     * @param messageSeqNumber The message sequence number, this should match a previously received HubMessage.
     * @param errorCode The message delivery status detail.
     */
    @EnforcePermission("ACCESS_CONTEXT_HUB")
    void sendMessageDeliveryStatus(int sessionId, int messageSeqNumber, byte errorCode);
}