Closed
Bug 823665
Opened 11 years ago
Closed 11 years ago
Rename urlclassifier.confirm-age for Firefox 18 and up
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
People
(Reporter: akeybl, Assigned: dcamp)
Details
(Whiteboard: [qa-])
Attachments
(1 file)
7.30 KB,
patch
|
gcp
:
review+
akeybl
:
approval-mozilla-aurora+
akeybl
:
approval-mozilla-beta+
akeybl
:
approval-mozilla-esr17+
|
Details | Diff | Splinter Review |
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•11 years ago
|
||
Attachment #694611 -
Flags: review?(gpascutto)
Reporter | ||
Updated•11 years ago
|
status-firefox-esr17:
--- → affected
tracking-firefox-esr17:
--- → 18+
Reporter | ||
Comment 2•11 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•11 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..."
Updated•11 years ago
|
Attachment #694611 -
Attachment is patch: true
Comment 4•11 years ago
|
||
Comment on attachment 694611 [details] [diff] [review] rename to max-complete-age LGTM.
Attachment #694611 -
Flags: review?(gpascutto) → review+
Comment 7•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/4fc4a3cda7d2 https://hg.mozilla.org/integration/mozilla-inbound/rev/6546477eb93d
Updated•11 years ago
|
Comment 8•11 years ago
|
||
https://hg.mozilla.org/releases/mozilla-b2g18/rev/477f49078cdd
status-b2g18:
--- → fixed
Updated•11 years ago
|
Target Milestone: --- → Firefox 20
Comment 9•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/6546477eb93d
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Comment 10•11 years ago
|
||
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.
Description
•