Ever try calling alert() in rapid succession, and notice that the dialog takes forever to appear? I have. commonDialog.xul needs to get spanked.

Working on this... just by changing collapsed="true" to hidden="true" on the optional container boxes, I save about 60% of window open time. Hyatt reminded me that doing this is tricky because that will cause the textbox/checkbox bindings to load async when they are shown, so I have to be sure to unset hidden from an inline function call instead of from onload.

This patch cleans up the commonDialog xul and js, changes collapsed to hidden on the textbox containers, converts to use the dialog tag, and updates the dialog tag to support extra buttons.

r=/sr=jag with the bug pointer.

Comment on attachment 63887 [details] [diff] [review] patch 3.0 r=bryner

a=blizzard on behalf of drivers for the cropping fixes

looks like this is checked in. can it be resolved?

Asa: only dialog.xml was checked in, the rest of the patch hasn't landed yet.

Removing the 0.9.8+ since my memory of the discussion was that we only wanted part of the patch for 0.9.8 (and thought the rest could wait), and it looks like part of the patch was checked in (I'm assuming the part we decided). If I got confused here and there's still something that needs to go in for 0.9.8, please get it in sooner rather than later.

almost forgot about this - landed on trunk now

This checkin caused the regression in bug 123216.