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

Add notification if a download is still active #53

Closed
MatzeKitt opened this issue Apr 30, 2013 · 7 comments
Closed

Add notification if a download is still active #53

MatzeKitt opened this issue Apr 30, 2013 · 7 comments

Comments

@MatzeKitt
Copy link

If a download is active and you want to close the last tab in private mode, the download stops without finishing. Is there any possibility to add a notification as in the original private mode?

@Infocatcher
Copy link
Owner

@Infocatcher
Copy link
Owner

Oh, yes, this is just about closing windows:
http://hg.mozilla.org/mozilla-central/file/3fb2370f4871/browser/base/content/browser.js#l5945
And there is no API to cancel tab closing...

@MatzeKitt
Copy link
Author

What a pity...

@Infocatcher
Copy link
Owner

I reported this to Firefox developers:
https://bugzilla.mozilla.org/show_bug.cgi?id=867349

But seems like only extensions may be buggy with this...

Infocatcher added a commit that referenced this issue May 1, 2013
…s "last-pb-context-exiting"

(workaround for #53, part 1)
Infocatcher added a commit that referenced this issue May 1, 2013
Infocatcher added a commit that referenced this issue May 1, 2013
Infocatcher added a commit that referenced this issue May 1, 2013
(workaround for #53, part 4; also related to #36 in SeaMonkey)
Infocatcher added a commit that referenced this issue May 1, 2013
@Infocatcher
Copy link
Owner

I added some workarounds for this issue.
I still can't prevent tab closing (and only can override gBrowser.removeTab() to do this), but seems all works fine, if I just open new private tab instead. Looks a bit strange, but better than nothing.

And still showing built-in warnings for private windows...

Test version: private_tab-0.1.3pre-fx-sm.xpi (source)

Infocatcher added a commit that referenced this issue May 4, 2013
…nAboutClosingWindow()

(workaround for #53, part 6)
Infocatcher added a commit that referenced this issue May 4, 2013
Infocatcher added a commit that referenced this issue May 4, 2013
…rs from WindowsPreviewPerTab.jsm

(workaround for #53, part 8)
@Infocatcher
Copy link
Owner

And still showing built-in warnings for private windows...

Should be better now: I created wrapper around built-in warnAboutClosingWindow() to make all windows looks like non-private (and send "last-pb-context-exiting" notification myself, so all warnings should work correct).

Test version: private_tab-0.1.3pre2-fx-sm.xpi (source)

Infocatcher added a commit that referenced this issue May 4, 2013
SeaMonkey doesn't send "last-pb-context-exiting" notifications for now
(workaround for #53, part 9)
Infocatcher added a commit that referenced this issue Jun 3, 2013
@Infocatcher
Copy link
Owner

Seems to be fixed now. :)

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

No branches or pull requests

2 participants