Add min and max values to app update integer values configured by preferences as appropriate

RESOLVED FIXED in Firefox 52

Status

()

P2
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: rstrong, Assigned: rstrong)

Tracking

unspecified
mozilla52
Points:
---

Firefox Tracking Flags

(firefox52 fixed)

Details

Attachments

(2 attachments, 3 obsolete attachments)

There are already a few of these and I'm going to add a few more as appropriate to prevent accidental changes as well as add-ons from breaking app update.
Created attachment 8809193 [details] [diff] [review]
client patch
Attachment #8808901 - Attachment is obsolete: true
Attachment #8808902 - Attachment is obsolete: true
Created attachment 8809259 [details] [diff] [review]
Firefox prefs patch
Attachment #8809259 - Flags: review?(felipc)
Attachment #8809193 - Flags: review?(mhowell)
Comment on attachment 8809194 [details] [diff] [review]
test patch

There is a little unrelated cleanup in the patches as well.
Attachment #8809194 - Flags: review?(mhowell)
Attachment #8809193 - Flags: review?(mhowell) → review+
Attachment #8809194 - Flags: review?(mhowell) → review+
Comment on attachment 8809259 [details] [diff] [review]
Firefox prefs patch

The prefs patch isn't important for this effort so I'm going to move it to a new bug.
Attachment #8809259 - Attachment is obsolete: true
Attachment #8809259 - Flags: review?(felipc)
Filed bug 1316621 for the Firefox preference cleanup

Comment 10

2 years ago
Pushed by rstrong@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/8db63d1c00cf
client code for Bug 1315505 - Add min and max values to app update integer values configured by preferences as appropriate to prevent prefs from breaking app update. r=mhowell
https://hg.mozilla.org/integration/mozilla-inbound/rev/70b739aa2d6a
test code for Bug 1315505 - Add min and max values to app update integer values configured by preferences as appropriate to prevent prefs from breaking app update. r=mhowell

Comment 11

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/8db63d1c00cf
https://hg.mozilla.org/mozilla-central/rev/70b739aa2d6a
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox52: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.