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

Commit 6a864b55 authored by Winson's avatar Winson
Browse files

Document non-effective mode return of AppOps batch fetch 

The getPackagesForOps and getOpsForPackage APIs don't convert the
mode of the return ops to their effective state, which isn't obvious.

Bug: 119281847

Test: none necessary

Change-Id: I60dd26ab771e73f2893981550a71cdaa353056c2
parent 8197beee

core/java/android/app/AppOpsManager.java

+24 −0
Original line number Diff line number Diff line
@@ -4294,6 +4294,12 @@ public class AppOpsManager { 
    /**

     * Retrieve current operation state for all applications.

     *

     * The mode of the ops returned are set for the package but may not reflect their effective

     * state due to UID policy or because it's controlled by a different master op.

     *

     * Use {@link #unsafeCheckOp(String, int, String)}} or {@link #noteOp(String, int, String)}

     * if the effective mode is needed.

     *

     * @param ops The set of operations you are interested in, or null if you want all of them.

     * @hide

     */
@@ -4312,6 +4318,12 @@ public class AppOpsManager { 
    /**

     * Retrieve current operation state for all applications.

     *

     * The mode of the ops returned are set for the package but may not reflect their effective

     * state due to UID policy or because it's controlled by a different master op.

     *

     * Use {@link #unsafeCheckOp(String, int, String)}} or {@link #noteOp(String, int, String)}

     * if the effective mode is needed.

     *

     * @param ops The set of operations you are interested in, or null if you want all of them.

     * @hide

     */
@@ -4328,6 +4340,12 @@ public class AppOpsManager { 
    /**

     * Retrieve current operation state for one application.

     *

     * The mode of the ops returned are set for the package but may not reflect their effective

     * state due to UID policy or because it's controlled by a different master op.

     *

     * Use {@link #unsafeCheckOp(String, int, String)}} or {@link #noteOp(String, int, String)}

     * if the effective mode is needed.

     *

     * @param uid The uid of the application of interest.

     * @param packageName The name of the application of interest.

     * @param ops The set of operations you are interested in, or null if you want all of them.
@@ -4354,6 +4372,12 @@ public class AppOpsManager { 
     * Retrieve current operation state for one application. The UID and the

     * package must match.

     *

     * The mode of the ops returned are set for the package but may not reflect their effective

     * state due to UID policy or because it's controlled by a different master op.

     *

     * Use {@link #unsafeCheckOp(String, int, String)}} or {@link #noteOp(String, int, String)}

     * if the effective mode is needed.

     *

     * @param uid The uid of the application of interest.

     * @param packageName The name of the application of interest.

     * @param ops The set of operations you are interested in, or null if you want all of them.