Consider eliminating the lazy frame construction xul:listbox does

NEW
Unassigned

Status

()

Core
XUL
7 years ago
7 years ago

People

(Reporter: bz, Unassigned)

Tracking

Trunk
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Just filing this so it's in bugzilla, not in my head.

This may be less of an issue now that we block remote XUL, but the lazy frame stuff listboxes do has been a PITA in all sorts of ways.  Unfortunately, it may be needed for performance reasons...

At some point I asked enn about our listbox test coverage, and he said:

------------------------------------------------------------
I had written a test long ago for listboxes, but never checked it in. I think it had a couple of issues, quite possibly due to the lazy frame construction.

The script in http://mxr.mozilla.org/mozilla-central/source/toolkit/content/tests/widgets/xul_selectontrol.js can be used to test much of the api. That script is used by both the radiogroup and tabbox tests and listboxes use much the same api.  Pretty much just modify the test_radio.xul test to use listbox instead of radios. (this is all that my listbox test did).

But it didn't test the list box object apis which are more likely to break from frame construction changes. 
------------------------------------------------------------
You need to log in before you can comment on or make changes to this bug.