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]: WebKit / Linux support avif image format #32673

Closed
pinanks opened this issue Sep 18, 2024 · 4 comments · Fixed by #32815
Closed

[Bug]: WebKit / Linux support avif image format #32673

pinanks opened this issue Sep 18, 2024 · 4 comments · Fixed by #32815

Comments

@pinanks
Copy link

pinanks commented Sep 18, 2024

Version

1.47.1

Steps to reproduce

  1. Clone repo at https://github.com/pinanks/pw-img-webkit-linux
  2. Run ./generate-ss.sh

Expected behavior

render-with-cache-response-ss

Actual behavior

A blank screenshot

render-with-cache-response-ss

Additional context

I am trying to render HTML with assets such as CSS/images etc cached and using page.route to intercept and fulfill with cached response. Everything works fine on my local machine (Apple M3 Pro, OS: Sonoma 14.1, everything else is same) but it fails on a Linux container.

The issue is that the img is not rendered and I get a blank screenshot. I also tried to have base64 of image as src directly in the html but it didn't work as well.

The repro will generate two screenshots for both these cases.

Environment

System:
  OS: Docker image ubuntu:jammy
Binaries:
  Node: v20.17.0
  npm: v10.8.2
Playwright:
  @playwright/test: v1.47.1
@mxschmitt
Copy link
Member

We currently compile our Linux builds without avif support - we'll look into enabling it.

@mxschmitt mxschmitt changed the title [Bug]: Images not rendered on Linux with Webkit [Bug]: WebKit / Linux support avif image format Sep 18, 2024
@mxschmitt mxschmitt self-assigned this Sep 18, 2024
@pinanks
Copy link
Author

pinanks commented Sep 23, 2024

Cool. Thanks.

@mxschmitt
Copy link
Member

This should be fixed in #33229 and available in Canary tomorrow.

@tats-u
Copy link

tats-u commented Oct 31, 2024

Can the change by #3322 be easily applied to Windows?

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

Successfully merging a pull request may close this issue.

3 participants