Bug 1357705 (stylo-inidomutils)

[meta] Stylo: Investigate inIDOMUtils APIs used by DevTools

NEW
Unassigned

Status

()

Core
CSS Parsing and Computation
P5
normal
a month ago
3 days ago

People

(Reporter: jryans, Unassigned)

Tracking

(Depends on: 2 bugs, Blocks: 2 bugs, {meta})

Firefox Tracking Flags

(Not tracked)

Details

DevTools currently uses[1] the following APIs from inIDOMUtils[2], some of which may need Stylo specific support:

Already updated for Stylo:

* getCSSStyleRules (bug 1346256)
* getAllStyleSheet (bug 1322661)

Appears to use Gecko style objects, probably needs updating for Stylo:

* parseStyleSheet
* selectorMatchesElement

Appears to use shared style objects, but Stylo passes invalid data:

* getRuleLine (line is always set to 0)
* getRuleColumn (column is always set to 0)
* getRelativeRuleLine (line is always set to 0)

Appears to only query static style data, could be deferred:

* getCSSPseudoElementNames
* isValidCSSColor
* getCSSPropertyNames
* getCSSValuesForProperty
* getSubpropertiesForCSSProperty
* isInheritedProperty
* cssPropertyIsShorthand
* cssPropertySupportsType
* cssPropertyIsValid
* getCSSLexer

Appears to poke element state, seems correctly forwarded to Stylo:

* hasPseudoClassLock
* addPseudoClassLock
* removePseudoClassLock
* clearPseudoClassLocks

Seems unrelated to Stylo:

* getUsedFontFaces

[1]: http://searchfox.org/mozilla-central/search?q=%5CbDOMUtils%5Cb&case=true&regexp=true&path=devtools
[2]: http://searchfox.org/mozilla-central/source/layout/inspector/inIDOMUtils.idl
(Reporter)

Updated

a month ago
Priority: -- → P3
(Reporter)

Updated

a month ago
Depends on: 1350175
(Reporter)

Updated

a month ago
Priority: P3 → P2
(Reporter)

Comment 1

a month ago
Clearing priority for now, so it appears on :bholley's triage, but probably P2?
Priority: P2 → --
Priority: -- → P2
Looks like I missed a few originally.

Appears to use Gecko style objects, probably needs updating for Stylo:

* getSelectorCount
* getSelectorText
* getSpecificity

Seems unrelated to Stylo:

* colorToRGBA
(Reporter)

Updated

3 days ago
Alias: stylo-inidomutils
(Reporter)

Updated

3 days ago
Keywords: meta
Priority: P2 → P5
Summary: Stylo: Investigate inIDOMUtils APIs used by DevTools → [meta] Stylo: Investigate inIDOMUtils APIs used by DevTools
(Reporter)

Updated

3 days ago
Depends on: 1367615
(Reporter)

Updated

3 days ago
Depends on: 1367620
You need to log in before you can comment on or make changes to this bug.