1839075 - Wrong time duration in CopyDownloadFeature

Status: NEW

(Firefox for Android :: Downloads, defect, P2)

Description

[jackyzy823]

https://github.com/mozilla-mobile/firefox-android/blob/main/android-components/components/feature/downloads/src/main/java/mozilla/components/feature/downloads/temporary/CopyDownloadFeature.kt#L65

private val operationTimeoutMs by lazy { TimeUnit.MINUTES.toMinutes(1) } is 1 (Long) will be used in withTimeout as 1 ms.

which conflicts with descrption with a 1 second timeout to ensure a smooth UX.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:jonalmeida, could you have a look please?

For more information, please visit BugBot documentation.

Comment 2

[BMO Github Automation]

Attachment: [mozilla-mobile/firefox-android] Bug 1839075 - Corrects timeout duration for CopyDownloadFeature (#4737)

Comment 3

[Chris Peterson [:cpeterson]]

The Android team has not been keeping our P1 bug list up to date, so we're resetting all our P1 bugs to P2 to avoid signalling that we're actively working on bugs that we're not. The BMO documentation https://wiki.mozilla.org/BMO/UserGuide/BugFields#priority says P1 means "fix in the current release cycle" and P2 means "fix in the next release cycle or the following (nightly + 1 or nightly + 2)".

If you are actively working on this bug and expect to ship it in Fx 122 or 123, then please restore the priority back to P1.