1848119 - Remove unnecessary PageStyleActor#_expandRules

Status: RESOLVED FIXED

(DevTools :: Inspector, task)

Description

[Nicolas Chevobbe [:nchevobbe]]

It does look like this is not something we benefit from

This was added a long time ago (https://hg.mozilla.org/mozilla-central/rev/b14e58f0e3ab1ae9c1b5f7c59949e7063dc9817e) and it feels like this was needed because we were exposing a parentRule property on the form that was the actorID (https://hg.mozilla.org/mozilla-central/rev/b14e58f0e3ab1ae9c1b5f7c59949e7063dc9817e#l2.579)
So the client would be able to get the information it needed through that property and needed to have the parent rule front.
We don't have such property anymore on the front (removed in Bug 1752313), and so it looks like it should be safe to not retrieve the parent rules.

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1848119 - [devtools] Remove unnecessary PageStyleActor#_expandRules. r=#devtools-reviewers.

Depends on D184728

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/444aa2304c68
[devtools] Remove unnecessary PageStyleActor#_expandRules. r=devtools-reviewers,jdescottes.

Comment 3

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/444aa2304c68