Open Bug 645563 Opened 13 years ago Updated 1 year ago

Implement scrollbars using a single anonymous element, instead of nested XUL NAC

Categories

(Core :: Layout, defect)

defect

Tracking

()

People

(Reporter: ehsan.akhgari, Unassigned)

References

(Blocks 3 open bugs)

Details

(Keywords: perf, Whiteboard: [overhead:5K])

From bug 237842 comment 31:

> We could probably win quite a lot of performance for all kinds of things by
> creating a single anonymous element that implements a complete scrollbar
> (instead of the complex XBL binding we currently use), and simplify a lot of
> stuff too. We could give that element a proper non-element-based API and move
> the thumb without having to do reflow. I thought there was a bug for this but I
> can't find it.
OS: Mac OS X → All
Hardware: x86 → All
See Bug 793942.
As long as we can theme it, it would be a great performance improvement probably.
We're going to remove the XBL in Bug 1431246, replicating the markup that's currently in XBL anon content into XUL NAC. That doesn't change anything about the benefits from Comment 0 - just going to mark this as dependent since it'll move around where the DOM gets created.
Depends on: 1431246
Summary: Implement scrollbars using a single anonymous element, instead of XBL → Implement scrollbars using a single anonymous element, instead of nested XUL NAC
Implementing this should at least save the memory cost of scrollbars.css (https://bugzilla.mozilla.org/show_bug.cgi?id=1474793#c8), some styles out of minimal-xul.css, and also whatever the NAC uses.
Whiteboard: [overhead:5K]
Blocks: 1446829

Emilio, will non-native theming make this XUL scrollbar bug obsolete?

Flags: needinfo?(emilio)

No

Flags: needinfo?(emilio)
Severity: normal → S3

The severity field for this bug is relatively low, S3. However, the bug has 18 votes.
:dholbert, could you consider increasing the bug severity?

For more information, please visit auto_nag documentation.

Flags: needinfo?(dholbert)

The last needinfo from me was triggered in error by recent activity on the bug. I'm clearing the needinfo since this is a very old bug and I don't know if it's still relevant.

Flags: needinfo?(dholbert)
You need to log in before you can comment on or make changes to this bug.