Closed Bug 562679 Opened 14 years ago Closed 14 years ago

Land new Add-ons Manager on trunk

Categories

(Toolkit :: Add-ons Manager, defect)

defect
Not set
normal

Tracking

()

VERIFIED FIXED
mozilla1.9.3a5

People

(Reporter: whimboo, Assigned: mossop)

References

Details

(Whiteboard: [rewrite])

Attachments

(2 files)

Once all remaining P1 blockers (which I have added to the dependency list are done, we have to finally land the new Add-ons Manager code on trunk.

Dave, I believe you will work on that. I will set you as assignee for now.
Blocks: 562682
This is now landed, we have a perf regression on OSX that I will be looking into over the next couple of days and then backing out if I don't resolve it.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
(In reply to comment #1)
> This is now landed, we have a perf regression on OSX that I will be looking
> into over the next couple of days and then backing out if I don't resolve it.

Pretty smooth landing, for the size of it. Not to suggest the perf regression is unimportant, but it seems we gain more than we lose by keeping it on trunk, and keeping the perf regression to a followup (since nothing actually broke).
Dave, am I correct when I say we do not have to run a backout? Can we start to mark all bugs with [fixed-in-addonsmgr] in the whiteboard entry as fixed now?
(In reply to comment #3)
> Dave, am I correct when I say we do not have to run a backout? Can we start to
> mark all bugs with [fixed-in-addonsmgr] in the whiteboard entry as fixed now?

The current situation is that unless I have resolved the Ts regression by Saturday then I will have to backout everything. I could go either way on marking the bugs at this point, but obviously if I mark them all as fixed then I'll have to reopen them all if I backout.
Ok, that makes sense.
In the event that we need to backout we need to also update removed-files. This should do it I believe based on comparing the windows zip files pre and post landing.
Attachment #442874 - Flags: review?(robert.bugzilla)
Attachment #442874 - Flags: review?(robert.bugzilla) → review+
Backed out:

http://hg.mozilla.org/mozilla-central/rev/8a7a28c65fbd
http://hg.mozilla.org/mozilla-central/rev/1240ddeed985
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Depends on: 563187
Depends on: 563012
And need to update removed-files.in again now that we've relanded
Attachment #444574 - Flags: review?(robert.bugzilla)
Attachment #444574 - Flags: review?(robert.bugzilla) → review+
Comment on attachment 444574 [details] [diff] [review]
new removed-files.in patch (landed)

Pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/e5117ef37402
Attachment #444574 - Attachment description: new removed-files.in patch → new removed-files.in patch (landed)
Status: REOPENED → RESOLVED
Closed: 14 years ago14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.3a5
Looks like it will be sticky now. Nothing major appeared after its landing. The Ts regression is known and not related to our code. Lets mark this bug as verified now.
Status: RESOLVED → VERIFIED
Flags: in-testsuite-
Flags: in-litmus-
Whiteboard: [rewrite]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: