Comment 35 for bug 1981163

Revision history for this message
In , Mkmelin+mozilla (mkmelin+mozilla) wrote :

We get no window interface, so this throws: https://searchfox.org/mozilla-central/rev/bf8d5de8528036c09590009720bc172882845b80/toolkit/components/pdfjs/content/PdfStreamConverter.jsm#140

Adding ` && alwaysAskBeforeHandling` to this condition fixes things (apart from still getting js errors in console, and the what-to-do dialog saying the pdf is an HTML document).
https://searchfox.org/mozilla-central/rev/a2c26b9b49a521f4be39559ca1ca9c345a237c70/toolkit/components/pdfjs/content/PdfStreamConverter.jsm#1145

Likewise a `getDOMWindow(aChannel, triggeringPrincipal);` call there, causing us to bail, will also fix it.

Bug 1726501 seems very related.
:Gijs, you worked on this for Firefox. Thoughts?