Closed Bug 1021031 Opened 7 years ago Closed 6 years ago

Remove temporary migration code added in bug 1019885

Categories

(Toolkit :: Password Manager, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla33
Tracking Status
firefox32 --- fixed
firefox33 --- fixed

People

(Reporter: Paolo, Assigned: Paolo)

References

Details

Attachments

(1 file)

The temporary migration code added in bug 1019885 should be removed once it had occasion to run on Nightly.
Attached patch The patchSplinter Review
Time to remove the temporary migration code.
Attachment #8441360 - Flags: review?(dolske)
Comment on attachment 8441360 [details] [diff] [review]
The patch

Review of attachment 8441360 [details] [diff] [review]:
-----------------------------------------------------------------

Presumably we'll want to remove this code in Aurora too, since it's been uplifted.
Attachment #8441360 - Flags: review?(dolske) → review+
Comment on attachment 8441360 [details] [diff] [review]
The patch

(In reply to Justin Dolske [:Dolske] from comment #2)
> Presumably we'll want to remove this code in Aurora too, since it's been
> uplifted.

Agreed, requesting uplift approval (you're listed as a release driver so I believe a redundant full uplift comment isn't needed here).
Attachment #8441360 - Flags: approval-mozilla-aurora?
Comment on attachment 8441360 [details] [diff] [review]
The patch

Aurora approval granted.
Attachment #8441360 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/mozilla-central/rev/2d3a23589a04
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #7)
> https://hg.mozilla.org/releases/mozilla-aurora/rev/59fe7d610045

Thanks for checking this in!
You need to log in before you can comment on or make changes to this bug.