Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

pdf.js not loading PDF's within the browser #12089

Closed
kjozwiak opened this issue Nov 23, 2017 · 3 comments
Closed

pdf.js not loading PDF's within the browser #12089

kjozwiak opened this issue Nov 23, 2017 · 3 comments

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Nov 23, 2017

Test Plan

1. Visit https://bitcoin.org/bitcoin.pdf within the URL bar ,PDF should load
2. search for `BTC pdf` within Google and click on the first search result, pdf should be loaded

Description

When loading a PDF directly into the URL, you'll receive a Your file was not found page error and when clicking on an external PDF link, you'll be asked to download the PDF rather than opening it within the browser.

Disabling/Enabling PDF.js from about:preferences#extensions and restarting the browser didn't help either.

Steps to Reproduce

  1. load https://bitcoin.org/bitcoin.pdf within the URL bar (PDF not being loaded)
  2. search for BTC pdf within Google and click on the first search result (should be the PDF)
  3. you'll be asked to download the PDF rather than opening it within the browser

Actual result:

PDF's are not being displayed/loaded within the Brave browser.

screen shot 2017-11-23 at 3 25 02 pm

Expected result:

PDF's should be loaded within the Brave browser.

Reproduces how often:

100% reproducible when using the above STR.

Brave Version

about:brave info:

Brave: 0.19.101
rev: 30c42b5
Muon: 4.5.18
libchromiumcontent: 63.0.3239.40

Reproducible on current live release:

Nope, looks like a C63 regression.

Additional Information

  • macOS 10.12.6 x64 - Reproduced
  • Win 10 x64 - Reproduced by @srirambv
  • Ununtu 17.10 x64 - Reproduced
@kjozwiak kjozwiak added 0.19.x issue first seen in 0.19.x cr63 plugin/pdfjs priority/P2 Crashes. Loss of data. Severe memory leak. regression release/blocking labels Nov 23, 2017
@kjozwiak kjozwiak added this to the 0.19.x + C63 (Release Channel) milestone Nov 23, 2017
@srirambv
Copy link
Collaborator

srirambv commented Dec 4, 2017

Seems to be fixed on Windows for 0.19.103. PDF loads properly

@srirambv srirambv closed this as completed Dec 4, 2017
@diracdeltas
Copy link
Member

fixed by brave/muon@fd36d1a

@Corporate-Solutions
Copy link

Persistent issue in Android. Just downloaded Brave to tablet two days ago. No PDFs. As if reader is not native.

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

No branches or pull requests

5 participants