Open Bug 1815315 Opened 2 years ago Updated 2 years ago

ext-downloads on android does not support persisting events

Categories

(WebExtensions :: Android, defect, P3)

defect

Tracking

(Not tracked)

People

(Reporter: mixedpuppy, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [addons-jira])

The events need to be converted to support event pages. See bug 1815310 for more detail.

Relevant implementation details at https://bugzilla.mozilla.org/show_bug.cgi?id=1796183#c3

In particular:

... need to be updated to account for the possibility of being suspended:

Here are examples of how the desktop implementations were updated (plus unit tests):

The new unit test here was disabled on Android because the downloads API internals differ for desktop and Android. Android's GeckoView side does have an implementation of downloads (bug 1656336), but lacks integration in A-C and/or Fenix (bug 1538348).

The conversion itself is trivial, so it could be a good exercise in running extension tests and making small changes there. The testing of the downloads API would be much more involved, but if you're experienced with GV development it should be feasible to mock the download internals (via DownloadDelegate; see patches from bug 1656336), either as a xpcshell test or a mochitest

Blocks: 1538348

Marking as P3 instead of P2 because the downloads API is not even working yet in Fenix (GeckoView has parts, but A-C/Fenix is not hooked up).

Severity: -- → S3
Priority: -- → P3
Whiteboard: [addons-jira]
You need to log in before you can comment on or make changes to this bug.