Open Bug 1432533 Opened 7 years ago Updated 2 years ago

Avoid file.exists() --> read file antipattern in migrator code

Categories

(Firefox :: Migration, enhancement, P3)

enhancement

Tracking

()

People

(Reporter: dthayer, Unassigned)

References

Details

(Keywords: perf)

Created as a follow-up for Bug 862127. There's a number of places in out migrator code where we check if files exists before reading them, when the faster thing to do would be to just attempt to read them and throw if we can't. In some cases, we're using the file.exists check to determine whether to show UI or not, so we'll need to split out the UI checks from the program logic checks.
Priority: -- → P3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.