Fix stupid copy-paste error when deleting session files

RESOLVED FIXED in Firefox 53

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: JanH, Assigned: JanH)

Tracking

unspecified
Firefox 55
All
Android
Points:
---

Firefox Tracking Flags

(firefox52 wontfix, firefox53 fixed, firefox54 fixed, firefox55 fixed)

Details

Attachments

(2 attachments)

(Assignee)

Updated

2 years ago
Blocks: 1266594
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)

Comment 3

2 years ago
mozreview-review
Comment on attachment 8846404 [details]
Bug 1346171 - Part 1 - Check correct file before attempting do delete it.

https://reviewboard.mozilla.org/r/119446/#review121468
Attachment #8846404 - Flags: review?(s.kaspari) → review+

Comment 4

2 years ago
mozreview-review
Comment on attachment 8846405 [details]
Bug 1346171 - Part 2 - Test synchronous session file deletion on clearing history.

https://reviewboard.mozilla.org/r/119448/#review121470
Attachment #8846405 - Flags: review?(s.kaspari) → review+

Comment 5

2 years ago
Pushed by mozilla@buttercookie.de:
https://hg.mozilla.org/integration/autoland/rev/303b1628ce44
Part 1 - Check correct file before attempting do delete it. r=sebastian
https://hg.mozilla.org/integration/autoland/rev/0be73c1886dd
Part 2 - Test synchronous session file deletion on clearing history. r=sebastian
(Assignee)

Comment 6

2 years ago
Comment on attachment 8846404 [details]
Bug 1346171 - Part 1 - Check correct file before attempting do delete it.

Approval Request Comment
[Feature/Bug causing the regression]: Bug 1266594
[User impact if declined]: When clearing history on shutdown, some sessions store files might not be cleaned up.
[Is this code covered by automated tests?]: Yes.
[Has the fix been verified in Nightly?]: No, but verified locally.
[Needs manual test from QE? If yes, steps to reproduce]:  No.
[List of other uplifts needed for the feature/fix]: none
[Is the change risky?]: No.
[Why is the change risky/not risky?]: Just changed check whether file exists to use the correct file.
[String changes made/needed]: none
Attachment #8846404 - Flags: approval-mozilla-beta?
Attachment #8846404 - Flags: approval-mozilla-aurora?
(Assignee)

Comment 7

2 years ago
Comment on attachment 8846405 [details]
Bug 1346171 - Part 2 - Test synchronous session file deletion on clearing history.

See Part 1 for the rest.
[List of other uplifts needed for the feature/fix]: Test modified here was introduced in bug 1343603 part 2, but that has already been uplifted to Aurora and Beta.
Attachment #8846405 - Flags: approval-mozilla-beta?
Attachment #8846405 - Flags: approval-mozilla-aurora?

Comment 8

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/303b1628ce44
https://hg.mozilla.org/mozilla-central/rev/0be73c1886dd
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox55: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
Comment on attachment 8846404 [details]
Bug 1346171 - Part 1 - Check correct file before attempting do delete it.

Fix for regression from 52, let's uplift to beta.
Attachment #8846404 - Flags: approval-mozilla-beta?
Attachment #8846404 - Flags: approval-mozilla-beta+
Attachment #8846404 - Flags: approval-mozilla-aurora?
Attachment #8846404 - Flags: approval-mozilla-aurora+
Comment on attachment 8846405 [details]
Bug 1346171 - Part 2 - Test synchronous session file deletion on clearing history.

Thanks for the test fixes.
Attachment #8846405 - Flags: approval-mozilla-beta?
Attachment #8846405 - Flags: approval-mozilla-beta+
Attachment #8846405 - Flags: approval-mozilla-aurora?
Attachment #8846405 - Flags: approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.