Closed Bug 1887638 Opened 2 months ago Closed 1 month ago

Add a (rust crate) cssparser based CSS lexer on InspectorUtils

Categories

(DevTools :: Inspector, task)

task

Tracking

(firefox127 fixed)

RESOLVED FIXED
127 Branch
Tracking Status
firefox127 --- fixed

People

(Reporter: nchevobbe, Assigned: nchevobbe)

References

Details

Attachments

(2 files)

We should re-use the same parser that stylo is using in the first place

This new InspectorCSSParser makes use of the cssparser crate so DevTools end
up using the same code as the CSS engine.

Assignee: nobody → nchevobbe
Status: NEW → ASSIGNED
Attachment #9393039 - Attachment description: Bug 1887638 - [devtools] Add InspectorUtils.getCSSParser method. r=emilio. → Bug 1887638 - [devtools] Add InspectorUtils.getCSSParser method. r=emilio,#devtools-reviewers.
Blocks: 1888353
Blocks: 1888363
Blocks: 1888441
Blocks: 1888607
Blocks: 1888609
Blocks: 1888615
Blocks: 1889363
Attachment #9393039 - Attachment description: Bug 1887638 - [devtools] Add InspectorUtils.getCSSParser method. r=emilio,#devtools-reviewers. → Bug 1887638 - [devtools] Add InspectorCSSParser method. r=emilio,#devtools-reviewers.
Attachment #9393039 - Attachment description: Bug 1887638 - [devtools] Add InspectorCSSParser method. r=emilio,#devtools-reviewers. → Bug 1887638 - [devtools] Add InspectorCSSParser. r=emilio,#devtools-reviewers.
Blocks: 1889871
Blocks: 1889875
Blocks: 1889920
Blocks: 1889938
Blocks: 1890493
Blocks: 1890552

In some cases, it can be more useful to only get the token value than the whole
token text (e.g. for 'Function`, where the value is the function name, while
the text includes the opening parenthesis)

Pushed by nchevobbe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d4cedbc852b9
[devtools] Add InspectorCSSParser. r=emilio,devtools-reviewers,frontend-codestyle-reviewers,bomsy.
https://hg.mozilla.org/integration/autoland/rev/978684d1d873
[devtools] Expose CSS token value in InspectorCSSToken. r=emilio.
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
Target Milestone: --- → 127 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: