Closed
Bug 1858305
Opened 2 years ago
Closed 2 years ago
Computed variable can be added without parsing
Categories
(Core :: CSS Parsing and Computation, task)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
121 Branch
| Tracking | Status | |
|---|---|---|
| firefox121 | --- | fixed |
People
(Reporter: zrhoffman, Assigned: zrhoffman)
References
Details
Attachments
(3 files)
In bug 1846632, a new variable is created by parsing the CSS string of the properties_and_values::ComputedValue.
Because we already know that the string is valid CSS, we can skip parsing and create a variable by providing the first and last token, the CSS string, and no references.
| Assignee | ||
Comment 1•2 years ago
|
||
WIP status because https://github.com/servo/rust-cssparser/pull/369
is not reviewed or merged.
| Assignee | ||
Comment 2•2 years ago
|
||
Depends on D193547
Updated•2 years ago
|
Assignee: nobody → zach
Status: NEW → ASSIGNED
| Assignee | ||
Comment 3•2 years ago
|
||
Depends on D193548
Updated•2 years ago
|
Attachment #9363494 -
Attachment description: WIP: Bug 1858305 - Update cssparser r=#supply-chain-reviewers,#style → Bug 1858305 - Update cssparser r=#supply-chain-reviewers,#style
Pushed by zach@zrhoffman.net:
https://hg.mozilla.org/integration/autoland/rev/96d0600a10ad
Update cssparser r=firefox-style-system-reviewers,emilio
https://hg.mozilla.org/integration/autoland/rev/194d0db1a1cb
Rewrite use of deprecated function TokenSerializationType::nothing() r=firefox-style-system-reviewers,emilio
Pushed by zach@zrhoffman.net:
https://hg.mozilla.org/integration/autoland/rev/a9fd21fa2333
Store ComputedPropertyValue without parsing r=firefox-style-system-reviewers,emilio
Comment 6•2 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/96d0600a10ad
https://hg.mozilla.org/mozilla-central/rev/194d0db1a1cb
https://hg.mozilla.org/mozilla-central/rev/a9fd21fa2333
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox121:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch
Updated•2 years ago
|
Flags: needinfo?(emilio)
Pushed by emilio@crisal.io:
https://hg.mozilla.org/integration/autoland/rev/82c83f9005e8
Fix supply-chain audits. a=bholley
Updated•2 years ago
|
Flags: needinfo?(emilio)
Comment 8•2 years ago
|
||
| bugherder | ||
You need to log in
before you can comment on or make changes to this bug.
Description
•