When a frame with a view is being created its view's visibility does not seem to be getting set. There is a hack in nsFrameControlFrame in DidReflow() that always sets the visibility to show. This completely breaks visibility for all form elements. I believe the fix is to move this code into Frame::DidReflow() and make it set the views visibility according to the style context. But this might be over kill. Another alternative is to do it when the view is initialized. Or should the style system be doing this somewhere? I'm not sure what the right approach is. Where should this go?

Peter marked this bug as fixed on 5/19 -- presuming he checked in a code fix. Marking verified