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

Tab spawned from private tab back history not always private. #230

Closed
windwakr opened this issue Aug 18, 2016 · 3 comments
Closed

Tab spawned from private tab back history not always private. #230

windwakr opened this issue Aug 18, 2016 · 3 comments

Comments

@windwakr
Copy link

windwakr commented Aug 18, 2016

If I right click the back arrow in a private tab, then middle click one of the pages to open on in a new tab, sometimes it will open in a non private tab.

Tested using Private Tab 0.1.9.1 and Firefox 45 ESR with a new profile.

@windwakr
Copy link
Author

Just noticed #131, which is somewhat similar to this. I have enabled debugging and caught what it says when this happens:

[Private Tab] 19:13:609 Tab opened: New Tab (empty)
Inherit private state: undefined
[Private Tab] 19:13:617 Selected tab not yet loaded, wait
[Private Tab] 19:13:644 updateWindowTitle() Mozilla Firefox
[Private Tab] 19:13:667 updateTabsInTitlebar() => TabsInTitlebar._update(true)
[Private Tab] 19:13:668 Hide tracking protection icon
[Private Tab] 19:13:668 setWindowBusy(): true
[Private Tab] 19:13:670 setWindowBusy(): false
[Private Tab] 19:13:691 Tab restored, private attribute: false, label: GitHub - Infocatcher/Private_Tab: Adds private tabs, restartless extension for Firefox (20.0+) and SeaMonkey (2.17+)
[Private Tab] 19:13:745 updateDownloadPanel() => reinitialize download indicator

And this is what I get logged when it works properly and opens a private tab:

[Private Tab] 19:41:438 Tab opened: New Tab (empty)
Inherit private state: undefined
[Private Tab] 19:41:446 Selected tab not yet loaded, wait
[Private Tab] 19:41:466 updateWindowTitle() Mozilla Firefox
[Private Tab] 19:41:497 updateTabsInTitlebar() => TabsInTitlebar._update(true)
[Private Tab] 19:41:497 Hide tracking protection icon
[Private Tab] 19:41:498 setWindowBusy(): true
[Private Tab] 19:41:499 setWindowBusy(): false
[Private Tab] 19:41:520 Tab restored, private attribute: true, label: Search · private tab · GitHub
[Private Tab] 19:41:520 Make restored tab private
Only internal code is allowed to set the usePrivateBrowsing attribute bootstrap.js:3557:2
[Private Tab] 19:41:520 Set usePrivateBrowsing to true
Tab: Search · private tab · GitHub
[Private Tab] 19:41:529 PrivateTab:PrivateChanged + tab is selected => updateWindowTitle()
[Private Tab] 19:41:529 updateWindowTitle() Mozilla Firefox (Private Browsing)
[Private Tab] 19:41:549 updateTabsInTitlebar() => TabsInTitlebar._update(true)
[Private Tab] 19:41:637 updateDownloadPanel() => reinitialize download indicator
[Private Tab] 19:41:860 [patcher] gBrowser.setIcon(): isPrivate = true
[Private Tab] 19:41:861 PrivateBrowsingUtils.isWindowPrivate(): override to true

@Infocatcher
Copy link
Owner

Tab restored, private attribute: false

May be this is something with sessions mechanism... Looks like sometimes tab state isn't updated yet.

But also this may be fixed from "Tab opened: New Tab (empty)" side.
And following fix should help: f08cc14

Test version: private_tab-0.1.9.2pre5-fx-sm.xpi (source code).

@windwakr
Copy link
Author

Yeah, that seems to fix it.

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