932387 - Support Blob for PDF view activity

Status: RESOLVED INVALID

(Firefox :: PDF Viewer, defect)

Description

[:Harald Kirschner :digitarald]

Most Gaia apps with activities allow viewing URLs or Blobs [1]. PDF Viewer should follow that pattern and also allow both.

PDF Viewer currently fails for opening URLs with authentication. A workaround apps could would be loading the file into a Blob via XHR and sending that to the PDF Viewer. Without blob support there is no other way to open those PDFs.

This seems to be a low hanging fruit that would unblock some partners and make the PDF Viewer more versatile and better integrated with other apps.

[1]: https://github.com/mozilla-b2g/gaia/blob/master/apps/video/js/view.js#L37

Comment 1

[:Harald Kirschner :digitarald]

Brendan might have an idea on feasibility.

Comment 2

[:Harald Kirschner :digitarald]

Reference: Bug 789126 blocks loading authenticated URLs.

Comment 3

[:Harald Kirschner :digitarald]

Maybe Yury has an idea here as well.

Comment 4

[Brendan Dahl [:bdahl]]

Can't you just get a url to the blob and then open that?

Comment 5

[Yury Delendik (:yury)]

Attachment: test with mozActivity

I see the following when trying to open the page:

E/GeckoConsole( 3695): [JavaScript Error: "NS_ERROR_DOM_BAD_URI: Access to restricted URI denied" {file: "app://pdfjs.gaiamobile.org/content/build/pdf.js" line: 110}]
I/Gecko   ( 3695): NS_ERROR_DOM_BAD_URI: Access to restricted URI denied

Can app:// originated page/worker read blobs?

Comment 6

[:Harald Kirschner :digitarald]

We need this for any app that wants to show PDFs but doesn't have a public web URL.

Comment 7

[:Harald Kirschner :digitarald]

Jonas, this is another part where Web Activities need to be standardized. When an app allows `view`ing some type of file it should take a resource from either `url` or `blob`.

Comment 8

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

I think we should add the Blob support here first, rather than blocking on the standardization effort. That should be relatively easy since at worst the pdf viewer can just get a blob:-URL for the blob once it has received it.

I've also reached out to Ehsan from the WebAPI team who can help with the standardization. But we don't need to block on that.

I forget which team is responsible for the pdf viewer app. But please make sure this gets on their backlog. This should be relatively easy to do, so we could hopefully target v2.0 for this. Though it's definitely not a blocker.

Comment 9

[Bill Walker [:bwalker] [@wfwalker]]

(In reply to Jonas Sicking (:sicking) from comment #8)

> 
> I forget which team is responsible for the pdf viewer app. But please make
> sure this gets on their backlog. This should be relatively easy to do, so we
> could hopefully target v2.0 for this. Though it's definitely not a blocker.

bdahl is our tech lead for this work. I'll follow up with him

Comment 10

[Kelly Price]

Has there been any updates on this? I'm on 66.0.4 (about to update to 66.0.5) on Windows 10 and this breaks pulling PDFs off my company (Databank) own support portal. Chrome works in this regard.

Comment 11

[Brendan Dahl [:bdahl]]

@Kelly: This bug was about Web Activities which I don't think was every standardized. You may want to open a new bug with your problem.