Page MenuHomePhabricator

Bug 1709415 - Support -webkit-image-set as a parse-time alias to image-set(). r=karlcow,twisniewski
ClosedPublic

Authored by emilio on May 12 2021, 12:51 AM.
Referenced Files
Unknown Object (File)
Apr 5 2025, 2:59 AM
Unknown Object (File)
Jan 11 2025, 9:58 PM
Unknown Object (File)
Jan 4 2025, 7:56 PM
Unknown Object (File)
Dec 29 2024, 3:50 PM
Unknown Object (File)
Dec 23 2024, 2:15 PM
Unknown Object (File)
Nov 27 2024, 4:31 AM
Unknown Object (File)
Mar 22 2024, 1:25 PM
Unknown Object (File)
Jan 29 2023, 7:11 AM
Subscribers

Details

Summary

The webkit syntax is an strict subset of the modern one, so this should
be doable, and is the simplest.

If my reading of the WebKit code is correct it should also be the way
WebKit deals with this (except they restrict -webkit-image-set() syntax
artificially).

Diff Detail

Repository
rMOZILLACENTRAL mozilla-central
Lint
Lint Not Applicable
Unit
Tests Not Applicable
Build Status
Buildable 320212
Build 414095: arc lint + arc unit

Event Timeline

phab-bot changed the visibility from "Custom Policy" to "Public (No Login Required)".
phab-bot changed the edit policy from "Custom Policy" to "Restricted Project (Project)".
phab-bot removed a project: secure-revision.

I let @twisniewski review the Rust code.
but the tests look good to me.

Thanks a lot @emilio

This revision is now accepted and ready to land.May 12 2021, 1:20 AM

This revision requires a Testing Policy Project Tag to be set before landing. Please apply one of testing-approved, testing-exception-unchanged, testing-exception-ui, testing-exception-elsewhere, testing-exception-other. Tip: this Firefox add-on makes it easy!

Code analysis found 2 defects in the diff 438035:

  • 2 defects found by wptlint-gecko (Mozlint)

You can run this analysis locally with:

  • ./mach lint --warnings --outgoing

If you see a problem in this automated review, please report it here.

You can view these defects on the code-review frontend and on Treeherder.