Rename urlclassifier.confirm-age for Firefox 18 and up

RESOLVED FIXED in Firefox 18

Status

()

Firefox
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: akeybl, Assigned: dcamp)

Tracking

unspecified
Firefox 20
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox18+ fixed, firefox19+ fixed, firefox20+ fixed, firefox-esr1718+ fixed, b2g18 fixed)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
In case we need to hotfix Firefox 18 due to bug 820283, let's rename urlclassifier.confirm-age in Firefox 18 and up.
(Assignee)

Comment 1

5 years ago
Created attachment 694611 [details] [diff] [review]
rename to max-complete-age
Attachment #694611 - Flags: review?(gpascutto)
(Reporter)

Updated

5 years ago
status-firefox-esr17: --- → affected
tracking-firefox-esr17: --- → 18+
(Reporter)

Comment 2

5 years ago
Comment on attachment 694611 [details] [diff] [review]
rename to max-complete-age

Pre-approving for all branches, contingent upon r+.
Attachment #694611 - Flags: approval-mozilla-esr17+
Attachment #694611 - Flags: approval-mozilla-beta+
Attachment #694611 - Flags: approval-mozilla-aurora+
(Reporter)

Comment 3

5 years ago
(In reply to Alex Keybl [:akeybl] from comment #0)
> In case we need to hotfix Firefox 18 due to bug 820283, let's rename
> urlclassifier.confirm-age in Firefox 18 and up.

This should have read "In case we need to hotfix Firefox 17..."
Attachment #694611 - Attachment is patch: true
Comment on attachment 694611 [details] [diff] [review]
rename to max-complete-age

LGTM.
Attachment #694611 - Flags: review?(gpascutto) → review+
https://hg.mozilla.org/releases/mozilla-esr17/rev/723a7eb89042
status-firefox18: affected → fixed
status-firefox-esr17: affected → fixed
status-firefox19: affected → fixed
status-firefox20: affected → fixed
Target Milestone: --- → Firefox 20
https://hg.mozilla.org/mozilla-central/rev/6546477eb93d
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Tagging this [qa-] since this has coverage in-testsuite. Please add qawanted keyword and advise QA if there is some manual testing needed here.
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.