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

RESOLVED FIXED in Firefox 52

Status

()

Product:
Toolkit
Component:
Application Update
Type:
defect
Importance:
P2
normal
Status:
RESOLVED FIXED
Opened:
3 years ago
Updated:
3 years ago

People

(Reporter: rstrong, Assigned: rstrong)

Tracking

Version:
unspecified
Target:
mozilla52
Points:
---

Firefox Tracking Flags

(firefox52 fixed)

Details

Attachments

(2 attachments, 3 obsolete attachments)

client patch
21.21 KB, patch
mhowell
: review+
Details | Diff | Splinter Review
test patch
28.28 KB, patch
mhowell
: review+
Details | Diff | Splinter Review 
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.
Priority: -- → P2
Posted patch client patch — — Splinter Review
Attachment #8808901 - Attachment is obsolete: true
Attachment #8808902 - Attachment is obsolete: true
Posted patch Firefox prefs patch (obsolete) — — Splinter Review
Attachment #8809259 - Flags: review?(felipc)
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
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
https://hg.mozilla.org/mozilla-central/rev/8db63d1c00cf
https://hg.mozilla.org/mozilla-central/rev/70b739aa2d6a
Status: ASSIGNED → RESOLVED
Closed: 3 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.