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

This commit is contained in:
Antoan Angelov
2020-06-02 10:52:04 +00:00
committed by Android (Google) Code Review
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