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

[Bug]: pdf-viewer sometimes breaks for the partition and stays blank #40454

Closed
3 tasks done
t57ser opened this issue Nov 6, 2023 · 2 comments
Closed
3 tasks done

[Bug]: pdf-viewer sometimes breaks for the partition and stays blank #40454

t57ser opened this issue Nov 6, 2023 · 2 comments
Labels

Comments

@t57ser
Copy link
Contributor

t57ser commented Nov 6, 2023

Preflight Checklist

Electron Version

26.4.1

What operating system are you using?

Windows

Operating System Version

10.0.19042

What arch are you using?

x64

Last Known Working Electron version

No response

Expected Behavior

Pdf viewer should always show the pdf

Actual Behavior

The following issue was observed in an app:

  • app downloads pdf
  • creates object url
  • opens a new window with the url
  • the pdf viewer stays blank (only a black screen)
  • pdf viewer stays broken accross the partition

example code:

const blob = await fetch("https://www.w3.org/WAI/ER/tests/xhtml/testfiles/resources/pdf/dummy.pdf").then(res => res.blob());
const url = URL.createObjectURL(blob)
window.open(url)

It happens fairly frequent in the app. I tried reproducing this in pure electron and it can be reproduced too but it happens very very rarely, so the gist might not help.
I used the following code for test purposes:

const blob = await fetch("https://www.w3.org/WAI/ER/tests/xhtml/testfiles/resources/pdf/dummy.pdf").then(res => res.blob());
const url = URL.createObjectURL(blob)
for (var i = 0; i < 100; i++) { window.open(url); }

I have a strong feeling that it is also related to the load that electron is under, since it happens in the app way easier than in plain electron.

Testcase Gist URL

https://gist.github.com/t57ser/386ccff1527e798c3dd0e26180e5fe3b

Additional Information

No response

@t57ser t57ser added the bug 🪲 label Nov 6, 2023
@codebytere
Copy link
Member

Duplicate of #27121

@codebytere codebytere marked this as a duplicate of #27121 Nov 6, 2023
@codebytere codebytere closed this as not planned Won't fix, can't repro, duplicate, stale Nov 6, 2023
@t57ser
Copy link
Contributor Author

t57ser commented Nov 7, 2023

@codebytere in the linked issue it is mentioned that using a persistent partition fixes the issue. This is not the case here. The tests were done with persistent partitions

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

No branches or pull requests

2 participants