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

Code Independent Code Points (CICP) in PNG images not supported #3498

Open
1 task
svgeesus opened this issue Feb 7, 2025 · 6 comments
Open
1 task

Code Independent Code Points (CICP) in PNG images not supported #3498

svgeesus opened this issue Feb 7, 2025 · 6 comments

Comments

@svgeesus
Copy link

svgeesus commented Feb 7, 2025

Summary

Running this WPT https://wpt.live/png/cicp-chunk.html gives a fail in Ladybird.

Operating system

Linux

Steps to reproduce

  1. Go to https://wpt.live/png/cicp-chunk.html

Expected behavior

Read "pass"

Other browsers support this, see
https://wpt.fyi/results/png?label=master&product=chrome%5Bexperimental%5D&product=edge%5Bexperimental%5D&product=firefox%5Bexperimental%5D&product=safari%5Bexperimental%5D&product=ladybird

Actual behavior

Read "fail"

URL for a reduced test case

https://wpt.live/png/cicp-chunk.html

HTML/SVG/etc. source for a reduced test case

n/a

Log output and (if possible) backtrace

n/a

Screenshots or screen recordings

No response

Build flags or config settings

No response

Contribute a patch?

  • I’ll contribute a patch for this myself.
@svgeesus
Copy link
Author

svgeesus commented Feb 7, 2025

Specification link: https://w3c.github.io/png/#cICP-chunk

CICP is very small (4 bytes) compared to a typical RGB ICC profile (5-20k) so usage is increasing.

@LucasChollet
Copy link
Contributor

Support for ICC profiles in Ladybird is globally pretty shallow, but that should be fairly easy to add. I'll try to submit a patch soon.

@trflynn89
Copy link
Contributor

@LucasChollet I suspect we can close this issue now, or was there more cICP work you wanted to do first?

Image

@LucasChollet
Copy link
Contributor

All cICP tests are passing but we only support the subset that makes the tests pass. That plus I wanted to import png/cicp-chunk.html with the import script and that's currently broken.
I can open new issues for these two items if you prefer.

@trflynn89
Copy link
Contributor

Gotcha, using this same issue is fine, just wanted to check!

@svgeesus
Copy link
Author

Web Platform Tests is now showing Ladybird passing

Image

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

No branches or pull requests

3 participants