Add API to unregister an update timer

RESOLVED WONTFIX

Status

()

Toolkit
Application Update
RESOLVED WONTFIX
5 years ago
5 years ago

People

(Reporter: Margaret, Assigned: Margaret)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

5 years ago
For bug 964447, I want to let a Fenenc add-on register to be notified of when it should update some data, such as once per day. We definitely can't count on Fennec to be running for 24 hours, so I want to use the update timer service. However, if a user uninstalls this add-on, we need to be able to unregister this timer, and right now we can't do this.
(Assignee)

Updated

5 years ago
Assignee: nobody → margaret.leibovic
Why not let the add-on itself register the timer so it will no longer be registered after it is uninstalled? Is it due to the add-on being restartless or something else?
(Assignee)

Comment 2

5 years ago
(In reply to Robert Strong [:rstrong] (use needinfo to contact me) from comment #1)
> Why not let the add-on itself register the timer so it will no longer be
> registered after it is uninstalled? Is it due to the add-on being
> restartless or something else?

Yes, my main concern was with restartless add-ons, which is generally what we encourage/support on Fennec. However, I was also concerned about the pref for that timer hanging around forever, but maybe that's not worth worrying about?
I'm not terribly worried about the pref but we can probably come up with a way to iterate them and clean up the ones that don't have registered timers without adding a new api.
(Assignee)

Comment 4

5 years ago
I actually came up with a solution that allows us to only register one timer ourselves, so we don't need to do this :)
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.