Closed Bug 1703319 Opened 4 years ago Closed 4 years ago

Determine whether the SessionStore "SHistoryListenerForRestore" is needed

Categories

(Firefox :: Session Restore, task)

Product:
Firefox
Component:
Session Restore
Type:
task

Tracking

()

Status:
RESOLVED INVALID

People

(Reporter: u608768, Assigned: u608768)

References

(Blocks 1 open bug)

Details

See https://searchfox.org/mozilla-central/rev/d7e344e956d9da2808ea33e1fe0f963ed10c142d/browser/components/sessionstore/SessionStore.jsm#5894-5899.

This was added in bug 1673617 to mirror the behaviour of ContentRestore's _historyListener.

Searchfox coverage info indicates that both the SHIP and non-SHIP paths are not instrumented, so we should determine if this is actually needed for anything.

This is needed. One test that relies on this is browser/components/sessionstore/test/browser_586068-reload.js. Not sure why coverage shows it's not instrumented.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INVALID
Assignee: nobody → kmadan
You need to log in before you can comment on or make changes to this bug.