Titles assigned to wrong windows after session restore about firetitle

[captnfab]

@eceprof hehe, thanks!
The addons.mozilla.org version has been approved. Please tell me if it works well for you :)

[captnfab]

Yes. Windows naming on restore is broken. Any help is welcome.

[eceprof]

Captnfab, you are fabulous. I just "upgraded" to Firefox Quantum and to my dismay lost the ability to use FireTitle, felt saved by Crappy FireTitle and then discovered the session restore problem!!! Imagine my relief when I opened this page and discovered at the end of it that you have found a solution the very day I looked for one. Thank you so much for being a lifesaver. FireTitle is second only to session restore to being my most needed and loved FireFox extension/capability.

[captnfab]

Thanks for your report, I'll investigate.

[captnfab]

Ok, I found the cause of the problem. When a session is restored, the windows are restored but their Id may change.
So, storing window's names / title patterns using window's id can't work.
Apparently, window.sessionId is the way to go, but those look empty…

[rsalmei]

Hey! Thanks for the response in the mozilla review.
Just came here to vote for the issue. It should have a way to accomplish this... Good luck!

[Betlista]

FF 57.0 forced me to switch from FireTitle to Crappy and after FF restart (I have option "Show your windows and tabs from last time") all titles (except one) are gone :-(

[Betlista]

@captnfab Can you point me directly to part, in which it is storing/loading titles?

[captnfab]

Sure. The problem is actually pretty simple.

Loadding occurs here:

FireTitle/firetitle/background.js

Line 8 in 106507e

cur_win_name = result["win"+tab.windowId+"_name"] || result.def_win_name || default_options["win_name"];

window's Pattern/name are saved in the local storage as win+windowId+_name/pattern. Saving part occurs here:

FireTitle/firetitle/background.js

Line 48 in 106507e

browser.storage.local.set(prop_set);

Unfortunately, window's ids change upon restore. This means storing parameters that way can't work.
When I tried, the window.sessionId was a blank field, so it was unreliable too.

Hence, the problem is I don't know how to keep track of a window through session restoration…

[captnfab]

I think that the local storage is the wrong way to go.
Using sessions seems a much clearer approach: https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/sessions

[captnfab]

Let me try to do something...

[Betlista]

I do not see the problem with local storage... The problem is a unique identifier. I see few candidates in https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/tabs/Tab id or sessionId, can you try tab.sessionId instead of window.sessionId ? I know that you are doing it on window level, maybe you can do something like save id/sessionId of a first tab in every window...

[captnfab]

I've got it working. I'm doing some cleaning since I'm not really proud of the state of the code.
I'm also polishing the UI.
I'll try to push the new version today and upload it to mozilla addons website for review.

[captnfab]

I've uploaded the new version to addons.mozilla.org and it is now pending review.

[Betlista]

I'm also going to test immediately ;-)

[Betlista]

Tested with 24 windows and 236 tabs and worked as expected, thanks for fix!