And in fact, we should create them when showing the dropdown, and tear them down when hiding the dropdown. Ehsan, please feel free to steal!

You asked for it! ;-) But I'm not sure if I can actually work on this before 2.0...

I think this is a post-2.0 project.

Is this about the widget created in nsCSSFrameConstructor::InitializeSelectFrame?

Yes.

This was much easier than I thought, which makes me think that I might have missed something! :-)

I think it would be slightly cleaner to put widget creation in ShowList alongside the destruction code.

Done.

Ehsan, in bug 631049 we are trying to prevent a widget ever being created for combobox dropdowns, for Fennec and Camino (which don't need such widgets). bz pointed me to your patch here, which removes the code changed in my patch there. Reading it now, I think the simplest thing would be to just change a single line in your patch and fix both bugs at once. Specifically, if + if (aShowList) { were + if (aShowList && !nsComboboxControlFrame::ToolkitHasNativePopup()) { then the widget will not be created if not needed (ToolkitHasNativePopup is true only on Fennec and Camino). What do you think?

(In reply to comment #8) > Ehsan, in bug 631049 we are trying to prevent a widget ever being created for > combobox dropdowns, for Fennec and Camino (which don't need such widgets). bz > pointed me to your patch here, which removes the code changed in my patch > there. Reading it now, I think the simplest thing would be to just change a > single line in your patch and fix both bugs at once. Specifically, if > > + if (aShowList) { > > were > > + if (aShowList && !nsComboboxControlFrame::ToolkitHasNativePopup()) { > > then the widget will not be created if not needed (ToolkitHasNativePopup is > true only on Fennec and Camino). What do you think? I'm usually a fan of smaller bugs, so I'd rather do that in bug 631049. I'm happy to own that bug too if you want me to (since I was the one who stepped on your toes there). :-)

Ok, as discussed on IRC I'll make the patch in the other bug fit on top of this one. As I was doing that I noticed that the patch here doesn't compile - nsComboboxControlFrame::ShowList uses view and viewManager which are not defined.

I guess code being compiled is a prerequisite for what we accept on m-c. This patch meets that criteria at least! ;-)