Closed Bug 1995818 Opened 9 months ago Closed 6 months ago

Add session restore support for splitview tabs

Categories

(Firefox :: Session Restore, enhancement, P1)

enhancement

Tracking

()

VERIFIED FIXED
149 Branch
Tracking Status
firefox149 --- verified

People

(Reporter: sclements, Assigned: nsharpley)

References

(Depends on 1 open bug, Blocks 2 open bugs)

Details

(Whiteboard: [fidefe-splitview])

Attachments

(1 file)

Behavior is outlined in the blueprint .

Assignee: nobody → sfoster
Status: NEW → ASSIGNED
Assignee: sfoster → nsharpley
Priority: P2 → P1
Duplicate of this bug: 2006797
Attachment #9534679 - Attachment description: WIP: Bug 1995818 - Add Session Store support for Split View → Bug 1995818 - Add Session Store support for Split View r=#sessionstore-reviewers
Pushed by nsharpley@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/88146a3e5bc6 https://hg.mozilla.org/integration/autoland/rev/b1232b240fcf Add Session Store support for Split View r=sessionstore-reviewers,tabbrowser-reviewers,sthompson,sclements
Pushed by imoraru@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/8e03c42067cc https://hg.mozilla.org/integration/autoland/rev/1882d49daf54 Revert "Bug 1995818 - Add Session Store support for Split View r=sessionstore-reviewers,tabbrowser-reviewers,sthompson,sclements" for causing marionette test failures on test_splitview_restore.py.

Revert for causing marionette test failures on test_splitview_restore.py.

Flags: needinfo?(nsharpley)

also these bc failures

Pushed by nsharpley@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/fe217f27dfed https://hg.mozilla.org/integration/autoland/rev/6f7aafb2d39e Add Session Store support for Split View r=sessionstore-reviewers,tabbrowser-reviewers,sthompson,sclements
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → 149 Branch
Blocks: 2012815

Hello! We have tested the following with Firefox 149.0a1 (2026-01-27) on Windows 11, macOS 26 aarch and Ubuntu 24:

  • Split Views that are inside/outside groups are correctly preserved after an update, crash, or restart - note that bug 2012661 was encountered
  • Split Views that are inside/outside groups are correctly restored when using the restore session enabled by default or manually restoring
  • Split Views that are inside/outside groups are correctly restored across windows when using the Recently closed windows option
  • Split Views are correctly preserved when switching between multiple profiles
  • Split Views are not restored after closing the tabs separately and then using the Recently closed tabs option or Ctrl+ Shift+ T keyboard shortcut
  • Split Views are restored while was in a group that has been deleted and restored via the Recently closed tabs option or Ctrl+ Shift+ T keyboard shortcut
  • Split View are restore while was in a group that has been closed and then restored via List all tabs menu or Ctrl+ Shift+ T keyboard shortcut

Do you think there is anything else that should be tested?

Also, from what I saw inside the Blueprint document, the last two implementations (restored Split Views inside a group) were not planned for V1, but they have been landed. I believe this is fine, but I’d like to confirm. Thank you!

Thanks Alexandru for the thorough testing! The last two implementations regarding tab groups are fine to include. They came as a bonus side effect to this patch.

Flags: needinfo?(nsharpley)

(In reply to Nikki Sharpley (:nikkis) (she/her) from comment #10)

Thanks Alexandru for the thorough testing! The last two implementations regarding tab groups are fine to include. They came as a bonus side effect to this patch.

Thank you! Closing this.

Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-ver-done-c149/b148]
See Also: → 2014505
Regressions: 2031961
Depends on: 2036554
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: