nsListControlFrame, nsFileControlFrame, and nsComboBoxControlFrame all suffer from the same basic problem as nsSliderFrame: they implement listener interfaces directly but they are not refcounted objects. They should create intermediate event listener objects that contain a weak reference back to the frame they represent. They also should not override AddRef() and Release() See bug 21571 for details.
I completely agree with Steve's analysis of the problem and his proposed solution
I really only want to do this once, so I am going to wait until we have the nsGfxListControlFrame working.
GfxListbox probably won't be done until M14, so moving this to M14
changing to M14
changing to M15
This is fixed for GfxList and I am closing it out now.
Reopening because this was fixed for just GfxList and now needs to be fixed for for non Gfx Listbox
Anyone have suggestions for how to test this?
It's a code-level fix. Whoever reviewed it for Rod should verify.
there isn't anyway to test it,
Fair enough. Thanks Buster, Rods for your comments. Based on engineering comments, marking VERIFIED FIXED on: - LinuxRH62 2000-09-13-08-M18 Commercial - Win98 2000-09-13-08-M18 Mozilla - MacOS86 2000-09-13-04-M18 Commercial