Closed
Bug 1490493
Opened 6 years ago
Closed 6 years ago
Don't close session when restoring the same session in GeckoView
Categories
(GeckoView :: General, defect, P3)
Tracking
(geckoview62 wontfix, firefox-esr60 wontfix, firefox62 wontfix, firefox63 fixed, firefox64 fixed)
RESOLVED
FIXED
mozilla64
People
(Reporter: jchen, Assigned: jchen)
References
Details
Attachments
(2 files)
When GeckoView tries to restore the same session that it already has, it should do nothing instead of closing the session.
Assignee | ||
Comment 1•6 years ago
|
||
Restoring the same session that's already managed by the GeckoView instance should be a no-op.
Assignee | ||
Comment 2•6 years ago
|
||
Add some tests for all scenarios that GeckoView encounters when it is saving/restoring state.
Comment 3•6 years ago
|
||
Comment on attachment 9008228 [details] Bug 1490493 - 1. Return early if same session is being restored; r?droeh Dylan Roeh (:droeh) has approved the revision.
Attachment #9008228 -
Flags: review+
Comment 4•6 years ago
|
||
Comment on attachment 9008229 [details] Bug 1490493 - 2. Add tests for GeckoView saving/restoring state; r?droeh Dylan Roeh (:droeh) has approved the revision.
Attachment #9008229 -
Flags: review+
Pushed by nchen@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/8a0955e0457a 1. Return early if same session is being restored; r=droeh https://hg.mozilla.org/integration/mozilla-inbound/rev/ab31927770b8 2. Add tests for GeckoView saving/restoring state; r=droeh
Comment 6•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/8a0955e0457a https://hg.mozilla.org/mozilla-central/rev/ab31927770b8
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Comment 7•6 years ago
|
||
Jim, does this session bug affect Focus? Should we uplift to GV 62 or 63 Beta?
status-firefox62:
--- → wontfix
status-firefox-esr60:
--- → wontfix
status-geckoview62:
--- → ?
Flags: needinfo?(nchen)
Assignee | ||
Comment 8•6 years ago
|
||
This bug follows bug 1484001, so it should be uplifted to 63 but not GV 62.
Assignee | ||
Comment 9•6 years ago
|
||
Comment on attachment 9008228 [details] Bug 1490493 - 1. Return early if same session is being restored; r?droeh Approval Request Comment [Feature/Bug causing the regression]: Bug 1484001 [User impact if declined]: The app might become not functional after backgrounding then foregrounding [Is this code covered by automated tests?]: Yes [Has the fix been verified in Nightly?]: Yes [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?]: Very small patch to address the bug [String changes made/needed]: None
Attachment #9008228 -
Flags: approval-mozilla-beta?
Comment 10•6 years ago
|
||
Comment on attachment 9008228 [details] Bug 1490493 - 1. Return early if same session is being restored; r?droeh Low risk patch improving reliability of Fennec, approved for 63 Beta, thanks.
Attachment #9008228 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 11•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/c5c6422e5b94 https://hg.mozilla.org/releases/mozilla-beta/rev/5101b8689c73
Updated•5 years ago
|
Product: Firefox for Android → GeckoView
Updated•5 years ago
|
Target Milestone: Firefox 64 → mozilla64
You need to log in
before you can comment on or make changes to this bug.
Description
•