Closed Bug 1200226 Opened 7 years ago Closed 7 years ago
Site session data loss
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.
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)
Attachment #8656693 - Flags: review?(rnewman)
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?
I think this is the right thing to do.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.