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][PTO - back on October 21]]

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][PTO - back on October 21]]

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