Merge "Only add user id to content uri if launching cross-profile target" into rvc-dev am: 6d5e349bcd

Original change: undetermined

Change-Id: Ic17cdd26febc197fc6deb3c465fb01e578ac63bb
This commit is contained in:
Antoan Angelov
2020-06-02 10:56:52 +00:00
committed by Automerger Merge Worker
2 changed files with 13 additions and 2 deletions

View File

@@ -3023,6 +3023,10 @@ public class ChooserActivity extends ResolverActivity implements
currentRootAdapter.updateDirectShareExpansion(); currentRootAdapter.updateDirectShareExpansion();
} }
void prepareIntentForCrossProfileLaunch(Intent intent) {
intent.fixUris(UserHandle.myUserId());
}
/** /**
* Adapter for all types of items and targets in ShareSheet. * Adapter for all types of items and targets in ShareSheet.
* Note that ranked sections like Direct Share - while appearing grid-like - are handled on the * Note that ranked sections like Direct Share - while appearing grid-like - are handled on the

View File

@@ -1235,8 +1235,8 @@ public class ResolverActivity extends Activity implements
} }
if (target != null) { if (target != null) {
if (intent != null) { if (intent != null && isLaunchingTargetInOtherProfile()) {
intent.fixUris(UserHandle.myUserId()); prepareIntentForCrossProfileLaunch(intent);
} }
safelyStartActivity(target); safelyStartActivity(target);
@@ -1250,6 +1250,13 @@ public class ResolverActivity extends Activity implements
return true; return true;
} }
void prepareIntentForCrossProfileLaunch(Intent intent) {}
private boolean isLaunchingTargetInOtherProfile() {
return mMultiProfilePagerAdapter.getCurrentUserHandle().getIdentifier()
!= UserHandle.myUserId();
}
@VisibleForTesting @VisibleForTesting
public void safelyStartActivity(TargetInfo cti) { public void safelyStartActivity(TargetInfo cti) {
// We're dispatching intents that might be coming from legacy apps, so // We're dispatching intents that might be coming from legacy apps, so