Closed Bug 1140672 Opened 7 years ago Closed 7 years ago

Remove app update file migration code for Windows

Categories

(Toolkit :: Application Update, defect)

x86_64
Windows 8.1
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed

People

(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)

Details

Attachments

(1 file)

This code has been around for over a year and a half and I think it should be removed.
Attached patch patch rev1Splinter Review
Assignee: nobody → robert.strong.bugs
Status: NEW → ASSIGNED
Note: it was added in bug 572162 for Firefox 24
Comment on attachment 8574238 [details] [diff] [review]
patch rev1

Looks good locally... pushed to try
https://treeherder.mozilla.org/#/jobs?repo=try&revision=04c7341a59bb

Stephen, this was "best effort" code to migrate the update metadata files when we moved the updates directory and it is as good of a time as any to remove it.
Attachment #8574238 - Flags: review?(spohl.mozilla.bugs)
Attachment #8574238 - Flags: review?(spohl.mozilla.bugs) → review+
Summary: Remove Windows update file migration code → Remove app update file migration code for Windows
Pushed to fx-team
https://hg.mozilla.org/integration/fx-team/rev/06abaa2883e1
Target Milestone: --- → mozilla39
You need to log in before you can comment on or make changes to this bug.