Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Private Tab may break in Firefox 28 (or later?) #82

Closed
Yoric opened this issue Jul 29, 2013 · 3 comments
Closed

Private Tab may break in Firefox 28 (or later?) #82

Yoric opened this issue Jul 29, 2013 · 3 comments

Comments

@Yoric
Copy link

Yoric commented Jul 29, 2013

Hi,

We are working on a number of optimizations to Session Restore for Firefox 25 (or 26). To perform one major optimization, we will need to get rid of the state string attached to "sessionstore-state-write" that you are currently using.

We need to find a way to ensure that this will not break Private Tab.

I have the feeling that if:

  • we keep sending "sessionstore-state-write" but without a nsISupportsString;
  • we don't save tabs that have attribute usePrivateBrowsing.

This will be sufficient to ensure that your add-on does not break and does not lose features. Could you confirm?

@Infocatcher
Copy link
Owner

Yes, I use "sessionstore-state-write" notification only to remove tabs with "privateTab-isPrivate" attribute from session data (because I don't have real tabs here). And only tabs with nsILoadContext.usePrivateBrowsing flag have that DOM attribute.

So, if you filter out all private tabs for me, I don't need "sessionstore-state-write" notification at all.

@Yoric
Copy link
Author

Yoric commented Aug 28, 2013

Let's do that.

@Infocatcher
Copy link
Owner

I updated the title to not confuse users:

-Private Tab may break in Firefox 25
+Private Tab may break in Firefox 28 (or later?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants