829611 - TabItem.prototype.getTabState looks horribly complex and expensive

Status: RESOLVED WONTFIX

(Firefox Graveyard :: Panorama, defect)

Description

[David Teller [:Yoric] - still alive but not very active]

If I understand correctly TabItem.prototype.getTabState, this method is designed for the sole purpose of returning the title and url of a tab.

However, barring any mistakes, here are a few side-effects of this function:
- serializing the history of the tab;
- storing all the current scroll positions of the frames inside the tab;
- serializing the session restore data for the tab (which might include SessionStorage, I'm not sure);
- deserializing aforementioned session restore data.

I believe that we should reimplement TabItem.prototype.getTabState to be a little more efficient.

Comment 1

[Justin Dolske [:Dolske]]

Panorama has been removed from Firefox 45, currently in Beta and scheduled for release on March 7th. As such, I'm closing all existing Panorama bugs.

If you are still using Panorama, you will see a deprecation message in Firefox 44, and when 45 is released your tab group data will be migrated to bookmarks, with a folder for each group. There are also a few addons offering similar functionality.

See https://support.mozilla.org/en-US/kb/tab-groups-removal for more info.

We're removing Panorama because it has extremely low usage (about 0.01% of users), and has a large number of bugs and usability issues. The cost of fixing all those issues is far too high to justify, and so we'll instead be focusing our time and energy on improving other parts of Firefox.