Closed Bug 847515 Opened 8 years ago Closed 3 years ago

Profile UI Performance to find bottlenecks and low hanging fruit for optimization


(Firefox for Metro Graveyard :: Theme, defect)

Windows 8
Not set


(Not tracked)



(Reporter: ally, Unassigned)



(Whiteboard: [defect] p=0)

We have a whole bunch of descendant selectors, which are very expensive and very bad for performance. 

We should try to purge platform.css of all of them.
The highest-priority ones are where the principal selector is very broad and does not use a class or ID, for example:

That one is really bad because the cost of matching the descendant selector will be incurred for every <description> element in the entire app.  When the principal selector is narrower, like this, the cost is much more limited and we should fix it only if it won't make our code much more fragile or complex:
Component: General → Theme
I think this is a good bug to do but I don't think it belongs in any story and I don't think it causes any human-noticeable jank, so I'm going to remove it from v1 triage. But feel free to work on it when you have time.
No longer blocks: metrov1triage
No longer blocks: metrov2planning
Whiteboard: [defect] p=0
I'm re-titling this bug. Changing selectors may or may not be a useful performance optimization, and may incur maintenance and reuse-potential costs. First we need to identify problems before prescribing fixes, so I suggest we use this bug for that work.
Summary: Death to Descendant Selectors → Profile UI Performance to find bottlenecks and low hanging fruit for optimization
Mass close of bugs in obsolete product
Closed: 3 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.