Skip to content
This repository has been archived by the owner on Oct 16, 2020. It is now read-only.

Avoid hiding and keeping track of private windows #112

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Avoid hiding and keeping track of private windows #112

wants to merge 1 commit into from

Conversation

keshto
Copy link

@keshto keshto commented Nov 5, 2013

Avoids adding the firetray Handler to windows that are private. This makes it so that when a user is surfing privately and exits the window it is not kept. Which would allow other users of the same machine to see the users session, which defeats the purpose of a private session.

@foudfou
Copy link
Owner

foudfou commented Nov 5, 2013

Nice! Thank you. Sounds reasonable. Could you add a pref for that (can't think of an obvious name right now) ? See 0f28d23 for how to do it.
You should also test under Thunderbird and Seamonkey, and think of other supported applications.

@@ -18,6 +18,29 @@ var firetrayChrome = { // each new window gets a new firetrayChrome !
winId: null,

onLoad: function(win) {
try {
// Firefox 20+
Components.utils.import("resource://gre/modules/PrivateBrowsingUtils.jsm");
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: Components.utils aliased to Cu

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

Successfully merging this pull request may close these issues.

2 participants