Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Change hasRequestRawExternalStorageAccess() to return integer

Browse Source 
Previously, hasRequestRawExternalStorageAccess would return null if the
app doesn't have requestRawExternalStorageAccess attribute in the
manifest. And, return true/false based on the value specified in
manifest.

Based on API review comments, changed the method to
getRequestRawExternalStorageAccess which returns
* RAW_EXTERNAL_STORAGE_ACCESS_DEFAULT if app didn't specify
requestRawExternalStorageAccess attribute in the manifest.
* RAW_EXTERNAL_STORAGE_ACCESS_REQUESTED if the app requested raw
external storage access.
* RAW_EXTERNAL_STORAGE_ACCESS_NOT_REQUESTED if the app requests to
disable raw external storage access

The API is not guarded with any system level API permissions, hence
changing the API to public API instead of system API. Also added
documentation to ensure apps don't misunderstand this API

Bug: 185484514
Test: atest packages/providers/MediaProvider
Change-Id: Ib7e41ab8ee38389bf44a360e4288d03e58ef44cf
This commit is contained in:
Sahana Rao
2021-04-27 17:13:42 +01:00
parent a7c3e1134b
commit d559e78253
3 changed files with 53 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
core/api/current.txt
View File

@@ -11921,6 +11921,7 @@ package android.content.pm {
method public int getGwpAsanMode();
method public int getMemtagMode();
method public int getNativeHeapZeroInitialized();
method public int getRequestRawExternalStorageAccess();
method public boolean isProfileable();
method public boolean isProfileableByShell();
method public boolean isResourceOverlay();
@@ -11976,6 +11977,9 @@ package android.content.pm {
field public static final int MEMTAG_DEFAULT = -1; // 0xffffffff
field public static final int MEMTAG_OFF = 0; // 0x0
field public static final int MEMTAG_SYNC = 2; // 0x2
field public static final int RAW_EXTERNAL_STORAGE_ACCESS_DEFAULT = 0; // 0x0
field public static final int RAW_EXTERNAL_STORAGE_ACCESS_NOT_REQUESTED = 2; // 0x2
field public static final int RAW_EXTERNAL_STORAGE_ACCESS_REQUESTED = 1; // 0x1
field public static final int ZEROINIT_DEFAULT = -1; // 0xffffffff
field public static final int ZEROINIT_DISABLED = 0; // 0x0
field public static final int ZEROINIT_ENABLED = 1; // 0x1

1
core/api/system-current.txt
View File

@@ -2520,7 +2520,6 @@ package android.content.om {
package android.content.pm {
public class ApplicationInfo extends android.content.pm.PackageItemInfo implements android.os.Parcelable {
method @Nullable public Boolean hasRequestRawExternalStorageAccess();
method public boolean isEncryptionAware();
method public boolean isInstantApp();
method public boolean isOem();

63
core/java/android/content/pm/ApplicationInfo.java
View File

@@ -2141,22 +2141,57 @@ public class ApplicationInfo extends PackageItemInfo implements Parcelable {
}
/**
* @return
* <ul>
* <li>{@code true} if this app requested raw external storage access
* <li>{@code false} if this app requests to disable raw external storage access.
* <li>{@code null} if the app didn't specify
* {@link android.R.styleable#AndroidManifestApplication_requestRawExternalStorageAccess}
* in its manifest file.
* </ul>
*
* Use default value for
* {@link android.R.styleable#AndroidManifestApplication_requestRawExternalStorageAccess}.
*/
public static final int RAW_EXTERNAL_STORAGE_ACCESS_DEFAULT = 0;
/**
* Raw external storage was requested by this app.
*/
public static final int RAW_EXTERNAL_STORAGE_ACCESS_REQUESTED = 1;
/**
* Raw external storage was not requested by this app.
*/
public static final int RAW_EXTERNAL_STORAGE_ACCESS_NOT_REQUESTED = 2;
/**
* These constants need to match the value of
* {@link android.R.styleable#AndroidManifestApplication_requestRawExternalStorageAccess}.
* in application manifest.
* @hide
*/
@SuppressWarnings("AutoBoxing")
@SystemApi
@Nullable
public Boolean hasRequestRawExternalStorageAccess() {
return requestRawExternalStorageAccess;
@IntDef(prefix = {"RAW_EXTERNAL_STORAGE_"}, value = {
RAW_EXTERNAL_STORAGE_ACCESS_DEFAULT,
RAW_EXTERNAL_STORAGE_ACCESS_REQUESTED,
RAW_EXTERNAL_STORAGE_ACCESS_NOT_REQUESTED,
})
@Retention(RetentionPolicy.SOURCE)
public @interface RawExternalStorage {}
/**
* @return
* <ul>
* <li>{@link ApplicationInfo#RAW_EXTERNAL_STORAGE_ACCESS_DEFAULT} if app didn't specify
* {@link android.R.styleable#AndroidManifestApplication_requestRawExternalStorageAccess}
* attribute in the manifest.
* <li>{@link ApplicationInfo#RAW_EXTERNAL_STORAGE_ACCESS_REQUESTED} if this app requested raw
* external storage access.
* <li>{@link ApplicationInfo#RAW_EXTERNAL_STORAGE_ACCESS_NOT_REQUESTED} if this app requests to
* disable raw external storage access
* </ul
* <p>
* Note that this doesn't give any hints on whether the app gets raw external storage access or
* not. Also, apps may get raw external storage access by default in some cases, see
* {@link android.R.styleable#AndroidManifestApplication_requestRawExternalStorageAccess}.
*/
public @RawExternalStorage int getRequestRawExternalStorageAccess() {
if (requestRawExternalStorageAccess == null) {
return RAW_EXTERNAL_STORAGE_ACCESS_DEFAULT;
}
return requestRawExternalStorageAccess ? RAW_EXTERNAL_STORAGE_ACCESS_REQUESTED
: RAW_EXTERNAL_STORAGE_ACCESS_NOT_REQUESTED;
}
/**