Open Bug 1836581 Opened 2 years ago Updated 2 years ago

Consolidate the ignorable tab URLs in SessionStore with the list in gInitialPages / isInitialPage from browser.js (e.g. move these to BrowserUtils)

Categories

(Firefox :: Session Restore, task)

Product:
Firefox
Component:
Session Restore
Type:
task

Tracking

()

People

(Reporter: sfoster, Unassigned)

References

Details

(Whiteboard: [fidefe-firefox-view])

We maintain a list of URLs in SessionStore that we use when determining if a tab's state is useful to save, including "about:blank", "about:newtab" etc. This is very similar to the functionality provided by isInitialPage from browser.js and the list in gInitialPages. We should be able to consolidate these to avoid the duplication.

Note that excluding "about:home" as a not-worth-saving tab URL causes this dontTemporarilyShowAboutHome test to fail

(In reply to Sam Foster [:sfoster] (he/him) from comment #1)

Note that excluding "about:home" as a not-worth-saving tab URL causes this dontTemporarilyShowAboutHome test to fail

I haven't fully grasped yet what's going on there, but if for some reason we don't want to use exactly the same list as browser.js, it seems like we should at least include chrome://browser/content/blanktab.html?

Depends on: 1676492
Summary: Consolidate the ignorable tab URLs in SessionStore with the list in gInitialPages / isInitialPage from browser.js → Consolidate the ignorable tab URLs in SessionStore with the list in gInitialPages / isInitialPage from browser.js (e.g. move these to BrowserUtils)
You need to log in before you can comment on or make changes to this bug.