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

Cherry-pick "LibWeb: Implement type attribute validation for image loading" #25021

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

circl-lastname
Copy link
Contributor

Cherry-picks LadybirdBrowser/ladybird#234
Generated with LBSync, did I do well?

This change fixes image loading where unsupported image types are
included in the list of source elements.

(cherry picked from commit a0561ea9b24c72489221079ee6ca20103bdbb231)
@github-actions github-actions bot added the 👀 pr-needs-review PR needs review from a maintainer or community member label Sep 13, 2024
@circl-lastname
Copy link
Contributor Author

circl-lastname commented Sep 13, 2024

Right, these should be derived from ImageLoader, and some of these we don't support

@circl-lastname circl-lastname marked this pull request as draft September 13, 2024 20:06
@github-actions github-actions bot removed the 👀 pr-needs-review PR needs review from a maintainer or community member label Sep 13, 2024
@nico
Copy link
Contributor

nico commented Sep 13, 2024

I remember intentionally not cherry picking this one a while ago, for the reasons you mention :)

@circl-lastname
Copy link
Contributor Author

Yak added..

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

Successfully merging this pull request may close these issues.

3 participants