Closed Bug 1851370 Opened 25 days ago Closed 13 days ago

[css-color] Vendor cssparser color parsing into Gecko

Categories

(Core :: CSS Parsing and Computation, task)

task

Tracking

()

RESOLVED FIXED
119 Branch
Tracking Status
firefox119 --- fixed

People

(Reporter: tlouw, Assigned: tlouw)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

Relative color syntax as well as color component serialization is coupled much closer with calc() values, e.g.

Relative color syntax: rgb(from blue r, calc(b / 2), g)
Component serialization: lab(calc(100) 0 0) should be serialized with the calc(..) function in place and not as 100.

Because cssparser does not have any concept of calc(), moving the code inside of Gecko where all the calc() code lives will make the parsing and serialization much easier.

Vendor the cssparser code and remove the abstractions that were required
to parse text into specified colors.

Assignee: nobody → tlouw
Attachment #9351378 - Attachment description: WIP: Bug 1851370 - Vendor cssparser::color::* parsing into style::color:parsing::* r=emilio,#layout → Bug 1851370 - Vendor cssparser::color::* parsing into style::color:parsing::* r=emilio,#layout
Status: NEW → ASSIGNED
Attachment #9351378 - Attachment is obsolete: true

To increase the integration between color and calc it is nescessary to
vendor the current cssparser-color library into Gecko where all the calc
functionality lives.

Pushed by tlouw@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/042192d18265
Vendor cssparser-color into Gecko r=emilio
Status: ASSIGNED → RESOLVED
Closed: 13 days ago
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
You need to log in before you can comment on or make changes to this bug.