Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

Focus crashes when downloading a media file on Google Pixel 6 (Android 12) #6346

Closed
delia-pop opened this issue Feb 3, 2022 · 2 comments
Closed
Assignees
Labels
Android 11/12 (FADP-8) Bring Focus up to date with changed functionality in Android bug crash 🔥 QAVerified
Milestone

Comments

@delia-pop
Copy link

Steps to reproduce

  1. Open a website with media files.
  2. Long tap on a link/imagine/file.
  3. Tap on "Download link"/"Save image".
  4. Observe the crash.

Expected behavior

The application remains stable at all times.

Actual behavior

The application crashes every time when trying to download a media file.

Device information

  • Android device: Google Pixel 6 (Android 12).
  • Focus version: Firefox Focus Nightly 98.0 (Build 360340507 🦎 98.0a1-20220202093701).
    Note that the crash was not encountered on other devices.
Focus.crashes.when.downloading.a.media.file.mp4
@delia-pop delia-pop added bug crash 🔥 Android 11/12 (FADP-8) Bring Focus up to date with changed functionality in Android labels Feb 3, 2022
@Mugurell
Copy link
Contributor

Mugurell commented Feb 4, 2022

Exception:

    java.lang.IllegalArgumentException: org.mozilla.focus.debug: Targeting S+ (version 31 and above) requires that one of FLAG_IMMUTABLE or FLAG_MUTABLE be specified when creating a PendingIntent.
    Strongly consider using FLAG_IMMUTABLE, only use FLAG_MUTABLE if some functionality depends on the PendingIntent being mutable, e.g. if it needs to be used with inline replies or bubbles.
        at android.app.PendingIntent.checkFlags(PendingIntent.java:375)
        at android.app.PendingIntent.getBroadcastAsUser(PendingIntent.java:645)
        at android.app.PendingIntent.getBroadcast(PendingIntent.java:632)
        at mozilla.components.feature.downloads.DownloadNotification.createPendingIntent(DownloadNotification.kt:292)
        at mozilla.components.feature.downloads.DownloadNotification.createDownloadCompletedNotification(DownloadNotification.kt:156)
        at mozilla.components.feature.downloads.AbstractFetchDownloadService.updateDownloadNotification$feature_downloads_release(AbstractFetchDownloadService.kt:374)
        at mozilla.components.feature.downloads.AbstractFetchDownloadService.updateDownloadNotification$feature_downloads_release$default(AbstractFetchDownloadService.kt:351)
        at mozilla.components.feature.downloads.AbstractFetchDownloadService.updateDownloadNotification(AbstractFetchDownloadService.kt:328)
        at mozilla.components.feature.downloads.AbstractFetchDownloadService.access$updateDownloadNotification(AbstractFetchDownloadService.kt:90)
        at mozilla.components.feature.downloads.AbstractFetchDownloadService$handleDownloadIntent$2.invokeSuspend(AbstractFetchDownloadService.kt:303)

@ionutbedregeanu Looks like the patch from here #6244 can be upstreamed to AC to then be used there and by all clients.

@delia-pop
Copy link
Author

Verified as fixed on Firefox Focus Nightly 99.0 (Build 360450513 🦎 99.0a1-20220212115333) with Google Pixel 6 (Android 12) and Motorola Moto G30 (Android 11).
The application remains stable when downloading a media file.

@delia-pop delia-pop added QAVerified and removed QAReady flag for test read & needed labels Feb 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android 11/12 (FADP-8) Bring Focus up to date with changed functionality in Android bug crash 🔥 QAVerified
Projects
None yet
Development

No branches or pull requests

2 participants