398941 - disable scrollbar XBL binding to test perf impact

Status: RESOLVED WONTFIX

(Toolkit :: UI Widgets, defect)

Description

[Robert Sayre]

scrollbar.xml shows up consistently high in dtrace profiles. We create this binding far more often than you might expect, because iframes, selects, etc. require one.

roc writes:

I wonder how much our scrollbar implementation costs us. Each
scrollbar instantiates an XBL binding which creates five anonymous
elements, each of which gets a frame and has other overhead. Every
<select> has one scrollbar, every <textarea> has two, every <iframe>
and top-level document has two, and every overflow:auto element has
two. It would be an interesting experiment to remove the XBL binding
and see what difference that makes to Tp. I'll do it next time I need
to play with Talos. If it's a significant improvement, we could build
a specialized XUL element + frame combination that renders a native
scrollbar.

Comment 1

[Colin Barrett [:cbarrett]]

FWIW roc's proposed change would massively help the mac scrollbar code. See bug 398137 and bug 385058 which could potentially be fixed by roc's changes.

I'd like to see it for 1.9.

Comment 2

[Mike Schroepfer]

Noming since this is a perf improvement (and maybe a big one)

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Would it be possible to attach some profiles here too? Or at least post some
numbers, not just "high in dtrace profiles".

Comment 4

[Robert Sayre]

Attachment: 10 loads of cnn.com

Comment 5

[Robert Sayre]

Comment on attachment 284685 [details]
10 loads of cnn.com

That's pretty simple page, and it doesn't show the cost of creating the xbl.

Comment 6

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Sayrer, did you do any testing here? Also, we should test after my fix for bug 384612 is in since that completely removes all scripts from scrollbars, and I think it should make us reflow less on mac as well.

Comment 7

[Mike Schroepfer]

Do we want to leave this bug open?

Comment 8

[Damon Sicore (:damons)]

This is not something that we're going to block beta 2 for, but we do need to get it done. Being realistic here and moving to a P2.  It's still on sayre's radar.

Comment 9

[Robert Sayre]

this won't make 1.9, moving off blocking list. renom if you disagree.

Comment 10

[Gregory Pappas [:gregp]]

Scrollbar XBL bindings were removed in bug 1431246