Kostas Reporter
	Description • 9 years ago

Using FF 44.0a1 x64.

STR: (in a clean profile)
- launch Firefox: it will open a Normal window
- open a Private window and open any PDF file, eg http://www.cbu.edu.zm/downloads/pdf-sample.pdf
Now, as it's displayed in the build-in PDF viewer (and it's completely loaded), 
click the 'down arrow' button in the build-in viewer in order to download it . 
Observe that:
1. the 'down arrow' button remains pressed
2. the animation in the Downloads button doesn't appear: if you clicki the button, you see that there's no entry for that file in the panel.
- Now, switch to the Normal window and click on the Downloads button:
you'll see that the entry for the PDF file was added there, instead.


Pn the other hand, if you download the PDF file 
via rightclicking on the document in the build-in PDF viewer and pressing 'Save Page as' instead, 
then the issue doesn't occur.

	Kostas Reporter
	Comment 1 • 9 years ago

This issue only occurs with e10s enabled, i.e. in e10s windows.

	Josh Matthews [:jdm]
	Comment 2 • 9 years ago

This sounds very similar to bug 1199841.

	:Felipe Gomes (needinfo for replies!) Assignee
	Comment 3 • 9 years ago

Kostas: you filed this bug just a few days ago. Did you notice whether this is a recent regression, or you can't tell for sure? (i.e., you used to do this often in the past and it worked fine, even with e10s on)

	Kostas Reporter
	Comment 4 • 9 years ago

No, unfortunately I can't tell for sure.

	Jim Mathies [:jimm]
	Comment 5 • 9 years ago

felipe, were you going to pick this up?

	:Felipe Gomes (needinfo for replies!) Assignee
	Comment 6 • 9 years ago

Yeah, and i'm hoping it's a dupe of bug 1199841, but we will see

	:Felipe Gomes (needinfo for replies!) Assignee
	Comment 7 • 9 years ago

The problem is that ExternalHelperAppParent implements nsIChannel but not PrivateBrowsingChannel, which makes nsExternalAppHandler::RetargetLoadNotifications miss the pb status when calling NS_UsePrivateBrowsing.

	Jason Duell
	Comment 8 • 9 years ago

Comment on attachment 8682054 [details] [diff] [review]
fix-externalhandler-pbbrowsing

Review of attachment 8682054 [details] [diff] [review]:
-----------------------------------------------------------------

I think this should work.  I assume you've tested the patch manually?

	:Felipe Gomes (needinfo for replies!) Assignee
	Comment 9 • 9 years ago

Yep I did, it works.  I'm assuming that there's nothing that needs to be changed here given the new patch in bug 8690005?

	:Felipe Gomes (needinfo for replies!) Assignee
	Comment 10 • 9 years ago

(In reply to :Felipe Gomes from comment #9)
> Yep I did, it works.  I'm assuming that there's nothing that needs to be
> changed here given the new patch in bug 8690005?

(Not sure how I came up with that number, but) I meant bug 1199841

	Pulsebot
	Comment 11 • 9 years ago

https://hg.mozilla.org/integration/fx-team/rev/bfd81e42d1dd

	Carsten Book [:Tomcat]
	Comment 12 • 9 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/bfd81e42d1dd

	Fahmida Noor
	Comment 13 • 8 years ago

I have reproduced this bug with Firefox Nightly 44.0a1 (Build Id : 20151001030236) on Windows 7(64-bit) with the instructions from comment 0.

Verified as fixed with Firefox Release 45.0.2 (Build ID : 20160407164938)

Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Firefox/45.0

[testday-20160415]