Closed
Bug 1011490
Opened 10 years ago
Closed 10 years ago
Use MozMap for InstallTrigger
Categories
(Toolkit :: Add-ons Manager, defect)
Toolkit
Add-ons Manager
Tracking
()
VERIFIED
FIXED
mozilla32
People
(Reporter: Gijs, Assigned: Gijs)
References
Details
(Whiteboard: p=2 s=it-32c-31a-30b.3 [qa-])
Attachments
(1 file)
3.87 KB,
patch
|
mossop
:
review+
bholley
:
review+
lmandel
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Per bug 1007878 comment #0.
Flags: needinfo?
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?
Assignee | ||
Comment 1•10 years ago
|
||
This was surprisingly easy. Thanks, Boris!
Attachment #8424119 -
Flags: review?(dtownsend+bugmail)
Assignee | ||
Updated•10 years ago
|
Attachment #8424119 -
Flags: review?(bobbyholley)
Assignee | ||
Comment 2•10 years ago
|
||
Try push: remote: https://tbpl.mozilla.org/?tree=Try&rev=2162992adb7f
Comment 3•10 years ago
|
||
Comment on attachment 8424119 [details] [diff] [review] use mozmap for installtrigger parameters, Review of attachment 8424119 [details] [diff] [review]: ----------------------------------------------------------------- Beautiful
Attachment #8424119 -
Flags: review?(dtownsend+bugmail) → review+
Comment 4•10 years ago
|
||
Comment on attachment 8424119 [details] [diff] [review] use mozmap for installtrigger parameters, Review of attachment 8424119 [details] [diff] [review]: ----------------------------------------------------------------- \o/
Attachment #8424119 -
Flags: review?(bobbyholley) → review+
Assignee | ||
Comment 5•10 years ago
|
||
I suck at tryserver syntax, so repushed as: remote: https://tbpl.mozilla.org/?tree=Try&rev=f040c2048e77 (see also: bug 1011700)
Assignee | ||
Comment 6•10 years ago
|
||
(In reply to :Gijs Kruitbosch from comment #5) > I suck at tryserver syntax, so repushed as: remote: > https://tbpl.mozilla.org/?tree=Try&rev=f040c2048e77 > > > (see also: bug 1011700) Hrmpf, test failures. Trivial fix is re-instating part of the manual arg checking I did. I'll land with that adjusted after confirming it fixes that test failure, now that bug 1007878 has landed.
Whiteboard: p=2
Assignee | ||
Comment 7•10 years ago
|
||
remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/8ceca2270d83
Status: NEW → ASSIGNED
Flags: firefox-backlog+
Whiteboard: p=2 → [fixed-in-inbound] p=2
Assignee | ||
Comment 8•10 years ago
|
||
Marco, this will need merging to m-c and uplift before we're done, so can you add this as pre-emptive carry-over to the upcoming iteration? Thanks!
Flags: needinfo?(mmucci)
Comment 9•10 years ago
|
||
Added to Iteration 32.2
Flags: needinfo?(mmucci)
Whiteboard: [fixed-in-inbound] p=2 → [fixed-in-inbound] p=2 s=it-32c-31a-30b.2 [qa?]
Updated•10 years ago
|
Whiteboard: [fixed-in-inbound] p=2 s=it-32c-31a-30b.2 [qa?] → [fixed-in-inbound] p=2 s=it-32c-31a-30b.3 [qa?]
Assignee | ||
Updated•10 years ago
|
Whiteboard: [fixed-in-inbound] p=2 s=it-32c-31a-30b.3 [qa?] → [fixed-in-inbound] p=2 s=it-32c-31a-30b.3 [qa-]
Comment 10•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8ceca2270d83
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
Updated•10 years ago
|
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 11•10 years ago
|
||
Comment on attachment 8424119 [details] [diff] [review] use mozmap for installtrigger parameters, [Approval Request Comment] Bug caused by (feature/regressing bug #): N/A User impact if declined: N/A Testing completed (on m-c, etc.): on m-c Risk to taking this patch (and alternatives if risky): low due to test coverage String or IDL/UUID changes made by this patch: none
Attachment #8424119 -
Flags: approval-mozilla-aurora?
Comment 12•10 years ago
|
||
Gijs - Given that this isn't a regression and from your approval request doesn't have user impact, can you please comment on the benefit of uplifting this change to Aurora?
Flags: needinfo?(gijskruitbosch+bugs)
Comment 14•10 years ago
|
||
Comment on attachment 8424119 [details] [diff] [review] use mozmap for installtrigger parameters, Aurora approval granted. Spoke with Gijs offline. This fix is beneficial to get in for the next ESR train.
Attachment #8424119 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Updated•10 years ago
|
Flags: needinfo?(gijskruitbosch+bugs)
Comment 15•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/092373432ba2
status-firefox31:
--- → fixed
status-firefox32:
--- → fixed
Whiteboard: [fixed-in-inbound] p=2 s=it-32c-31a-30b.3 [qa-] → p=2 s=it-32c-31a-30b.3 [qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•