Closed Bug 1421066 Opened 2 years ago Closed 2 years ago

getAppIniString in devtools/shared/system.js is slow

Categories

(DevTools :: Framework, enhancement, P2)

enhancement

Tracking

(firefox59 fixed)

RESOLVED FIXED
Firefox 59
Tracking Status
firefox59 --- fixed

People

(Reporter: gregtatum, Assigned: tromey)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

It hits the disk, and we're probably not even using that information. We should probably lazily instantiate it.

https://perfht.ml/2Abqc4o

I was on my home windows machine and noticed reaaaaally slow DevTools while installing some things in the background.
Priority: -- → P2
Assignee: nobody → ttromey
Comment on attachment 8932523 [details]
Bug 1421066 - remove getAppIniString from devtools/shared/system.js;

https://reviewboard.mozilla.org/r/203572/#review209024
Attachment #8932523 - Flags: review?(poirot.alex) → review+
Pushed by ttromey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ec355409f5e7
remove getAppIniString from devtools/shared/system.js; r=ochameau
https://hg.mozilla.org/mozilla-central/rev/ec355409f5e7
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 59
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.