[criticalAY]

Note

Assisted-by: Opus 5.5

Purpose / Description

Importing from the device storage failed with com.android.externalstorage has no access to ... on phones where the system External Storage app has lost "All files access", and each failure was sent to ACRA.

Fixes

• Fixes SecurityException: com.android.externalstorage has no access to content://media/external_primary/file/id #22822

Approach

On Android 12+, retry through MediaStore.getMediaUri, which reuses the access we already have to the picked file. If that also fails, the error is reported as before.

How Has This Been Tested?

CI and repro tests

Learning (optional, can help others)

Since Android 12 the storage provider reads files through MediaStore, which needs All files access, so every app fails the same way on these phones.

Checklist

Please, go through these checks before submitting the PR.

• You have a descriptive commit message with a short title (first line, max 50 chars).
• You have commented your code, particularly in hard-to-understand areas
• You have performed a self-review of your own code
• UI changes: include screenshots of all affected screens (in particular showing any new or changed strings)
• UI Changes: You have tested your change using the Google Accessibility Scanner

[david-allison]

Really sorry, could we block this on:

• fix(import): unify import logic & change prioritization of MIME/extension #22842

[david-allison]

Windows tests failing

[david-allison]

LGTM, thanks!