1911510 - Turn InplaceEditor cssVariables (Map) option into getCssVariables (Function) so it's only called when needed

Status: RESOLVED FIXED

(DevTools :: Shared Components, task)

Description

[Nicolas Chevobbe [:nchevobbe]]

In the Rules view, we go through the existing Maps for CSS variables to pass it to the inplace editor. Doing it upfront is costly, especially since we might need to create new Maps.
Turning the option into a function will allow "lazy loading" the CSS variable list.

Comment 1

[Nicolas Chevobbe [:nchevobbe]]

Attachment: Bug 1911510 - [devtools] Lazy load cssProperties in InplaceEditor. r=#devtools-reviewers.

Change the cssVariables option, a Map, into getCssVariables, a function,
so it can be called only when it's needed.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f8794323217a
[devtools] Lazy load cssProperties in InplaceEditor. r=devtools-reviewers,bomsy.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a33c19383e54
[devtools] Lazy load cssProperties in InplaceEditor. r=devtools-reviewers,bomsy.

Comment 4

[pstanciu]

https://hg.mozilla.org/mozilla-central/rev/a33c19383e54