From 9432f7d43c7ab21ff2905981df690ee7d0c50f1a Mon Sep 17 00:00:00 2001 From: Makoto Onuki Date: Wed, 9 Oct 2019 15:33:11 -0700 Subject: [PATCH] Add Context.createContextAsUser() Without it, apps (mainline modules) will need to use createPackageContext..., which is a bit painful. Bug: 142472686 Test: atest android.content.cts.ContextTest#testCreateContextAsUser Change-Id: Id640e03862462724df1a4a3101f0b08faafba22f (cherry picked from commit b844001d6a8f0b10b2f379fd5bcdea8e896e7de7) Merged-in: Id640e03862462724df1a4a3101f0b08faafba22f --- api/system-current.txt | 3 ++- api/test-current.txt | 3 ++- core/java/android/app/ContextImpl.java | 9 +++++++++ core/java/android/content/Context.java | 20 ++++++++++++++++++- core/java/android/content/ContextWrapper.java | 6 ++++++ .../src/android/test/mock/MockContext.java | 6 ++++++ 6 files changed, 44 insertions(+), 3 deletions(-) diff --git a/api/system-current.txt b/api/system-current.txt index a58cfd28f8ac5..a599a0f8fe825 100644 --- a/api/system-current.txt +++ b/api/system-current.txt @@ -1355,8 +1355,9 @@ package android.content { public abstract class Context { method @RequiresPermission(android.Manifest.permission.INTERACT_ACROSS_USERS) public boolean bindServiceAsUser(@RequiresPermission android.content.Intent, android.content.ServiceConnection, int, android.os.UserHandle); + method @NonNull public android.content.Context createContextAsUser(@NonNull android.os.UserHandle); method public abstract android.content.Context createCredentialProtectedStorageContext(); - method public android.content.Context createPackageContextAsUser(String, int, android.os.UserHandle) throws android.content.pm.PackageManager.NameNotFoundException; + method @NonNull public android.content.Context createPackageContextAsUser(@NonNull String, int, @NonNull android.os.UserHandle) throws android.content.pm.PackageManager.NameNotFoundException; method @Nullable public abstract java.io.File getPreloadsFileCache(); method public abstract boolean isCredentialProtectedStorage(); method public abstract void sendBroadcast(android.content.Intent, @Nullable String, @Nullable android.os.Bundle); diff --git a/api/test-current.txt b/api/test-current.txt index 18d9f75b5aae4..1c57ba6cee7ac 100644 --- a/api/test-current.txt +++ b/api/test-current.txt @@ -639,7 +639,8 @@ package android.content { } public abstract class Context { - method public android.content.Context createPackageContextAsUser(String, int, android.os.UserHandle) throws android.content.pm.PackageManager.NameNotFoundException; + method @NonNull public android.content.Context createContextAsUser(@NonNull android.os.UserHandle); + method @NonNull public android.content.Context createPackageContextAsUser(@NonNull String, int, @NonNull android.os.UserHandle) throws android.content.pm.PackageManager.NameNotFoundException; method public abstract android.view.Display getDisplay(); method public abstract int getDisplayId(); method public android.os.UserHandle getUser(); diff --git a/core/java/android/app/ContextImpl.java b/core/java/android/app/ContextImpl.java index 9902f6aa0fe37..c2e413c9f39a6 100644 --- a/core/java/android/app/ContextImpl.java +++ b/core/java/android/app/ContextImpl.java @@ -2223,6 +2223,15 @@ class ContextImpl extends Context { "Application package " + packageName + " not found"); } + @Override + public Context createContextAsUser(UserHandle user) { + try { + return createPackageContextAsUser(getPackageName(), mFlags, user); + } catch (NameNotFoundException e) { + throw new IllegalStateException("Own package not found: package=" + getPackageName()); + } + } + @Override public Context createContextForSplit(String splitName) throws NameNotFoundException { if (!mPackageInfo.getApplicationInfo().requestsIsolatedSplitLoading()) { diff --git a/core/java/android/content/Context.java b/core/java/android/content/Context.java index a12e4da6e23e3..6ce81d8e7b9ac 100644 --- a/core/java/android/content/Context.java +++ b/core/java/android/content/Context.java @@ -5256,8 +5256,9 @@ public abstract class Context { */ @SystemApi @TestApi + @NonNull public Context createPackageContextAsUser( - String packageName, @CreatePackageOptions int flags, UserHandle user) + @NonNull String packageName, @CreatePackageOptions int flags, @NonNull UserHandle user) throws PackageManager.NameNotFoundException { if (Build.IS_ENG) { throw new IllegalStateException("createPackageContextAsUser not overridden!"); @@ -5265,6 +5266,23 @@ public abstract class Context { return this; } + /** + * Similar to {@link #createPackageContext(String, int)}, but for the own package with a + * different {@link UserHandle}. For example, {@link #getContentResolver()} + * will open any {@link Uri} as the given user. + * + * @hide + */ + @SystemApi + @TestApi + @NonNull + public Context createContextAsUser(@NonNull UserHandle user) { + if (Build.IS_ENG) { + throw new IllegalStateException("createContextAsUser not overridden!"); + } + return this; + } + /** * Creates a context given an {@link android.content.pm.ApplicationInfo}. * diff --git a/core/java/android/content/ContextWrapper.java b/core/java/android/content/ContextWrapper.java index 2b1b8eb183ab2..f7a0eb84c9250 100644 --- a/core/java/android/content/ContextWrapper.java +++ b/core/java/android/content/ContextWrapper.java @@ -896,6 +896,12 @@ public class ContextWrapper extends Context { return mBase.createPackageContextAsUser(packageName, flags, user); } + /** @hide */ + @Override + public Context createContextAsUser(UserHandle user) { + return mBase.createContextAsUser(user); + } + /** @hide */ @Override @UnsupportedAppUsage diff --git a/test-mock/src/android/test/mock/MockContext.java b/test-mock/src/android/test/mock/MockContext.java index 727684eca7ea7..c7e536e8bafe8 100644 --- a/test-mock/src/android/test/mock/MockContext.java +++ b/test-mock/src/android/test/mock/MockContext.java @@ -763,6 +763,12 @@ public class MockContext extends Context { throw new UnsupportedOperationException(); } + /** {@hide} */ + @Override + public Context createContextAsUser(UserHandle user) { + throw new UnsupportedOperationException(); + } + /** {@hide} */ @Override public int getUserId() {