Closed Bug 1012231 Opened 10 years ago Closed 7 years ago

reset default prefs on addon unload

Categories

(Add-on SDK Graveyard :: General, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: zombie, Unassigned)

References

Details

SDK never did this, possibly because default pref branch can't be "reset" according to [1]: > Note: This method can be called on either a default or user branch but, > in effect, always operates on the user branch. but we could maybe delete the branch, if the branch name was not changed using preferences-branch. [1] https://developer.mozilla.org/en-US/docs/Mozilla/Tech/XPCOM/Reference/Interface/nsIPrefBranch#resetBranch%28%29
Flags: needinfo?(rFobic)
I don't really understand what the intent of deleting the branch on the add-on unload is, could you please elaborate ?
Flags: needinfo?(rFobic)
I think this should be a won't fix, it's useful to keep the prefs if/when the add-on is reinstalled.
this is about clearing *default* prefs added by the addon, not about prefs actually set by the *user*. old-school addons don't leave default prefs after they are uninstalled (after a restart). naturally, since we set them on addon startup, jetpack-set prefs are not persisted after a restart, but they are after addon disable/uninstall, which we can (and imho should) try to fix. also, this is probably related to bug 837023, though that one might also include interaction with user-set preferences branch.
Priority: -- → P2
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.