Closed Bug 1712642 Opened 2 years ago Closed 2 years ago

Support overriding a backup file that already exists in IOUtils.write

Categories

(Toolkit :: OS.File, defect)

Product:
Toolkit
Component:
OS.File
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
90 Branch
Tracking Flags:
Tracking Status
firefox-esr78 --- unaffected
firefox88 --- unaffected
firefox89 --- unaffected
firefox90 --- fixed

People

(Reporter: emmamalysz, Assigned: emmamalysz)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Bug 1712642: allow overwrite for WriteMode::Overwrite and WriteMode::Append
48 bytes, text/x-phabricator-request
Details | Review

In Bug 1663707, we introduced an incorrect check here: https://searchfox.org/mozilla-central/rev/16388511162ac6bbf5399e69e500e174a2f3972c/dom/system/IOUtils.cpp#935

We will need to be able to overwrite contents of an existing backup file.

Blocks: 1649604

Set release status flags based on info from the regressing bug 1663707

status-firefox88: --- → unaffected
status-firefox89: --- → unaffected
status-firefox90: --- → affected
status-firefox-esr78: --- → unaffected
Pushed by emalysz@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/6474cb0546bb
allow overwrite for WriteMode::Overwrite and WriteMode::Append r=barret

https://hg.mozilla.org/mozilla-central/rev/6474cb0546bb

Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox90: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.