Closed Bug 1200226 Opened 6 years ago Closed 6 years ago

Site session data loss

Categories

(Firefox for iOS :: General, defect)

All
iOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
fxios-v1.0 --- affected
fxios 1.0.5+ ---

People

(Reporter: aaronmt, Assigned: sleroux)

References

Details

Attachments

(2 files)

https://groups.google.com/a/mozilla.com/d/msgid/ios-mobile-testers/1441010325081.bf65a14f4f46e8%40mozgaia

It isn't clear, a) this only happens when the active session in a tab is closed b) on device idle (like the WKProcessPool reset bug) and c) on sites other than Bugzilla.
For this bug, we'll remove the workaround KVOURL code where we reset the process pool to resolve this issue.
Depends on: 1194726
Assignee: nobody → sleroux
This patch removes the resetProcessPool call from KVOURL and memory warning callbacks. We'll probably want to see what impact this change has on the white screen issue.
Attachment #8656691 - Flags: review?(rnewman)
v1.0 Patch
Attachment #8656693 - Flags: review?(rnewman)
Status: NEW → ASSIGNED
Comment on attachment 8656691 [details] [review]
https://github.com/mozilla/firefox-ios/pull/1030

I'm mildly concerned that we're changing two things at once here, but it's not a bad starting point if you're confident.
Attachment #8656691 - Flags: review?(rnewman) → review+
Attachment #8656693 - Flags: review?(rnewman) → review+
I think we should remove the resetting of the content pool from both because I feel like if we don't remove it from the memory callback, the issue might not be resolved in that case.
Any thoughts on this :st3fan?
Flags: needinfo?(sarentz)
I think this is the right thing to do.
Flags: needinfo?(sarentz)
Merged
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.