As a prerequisite for implementing the HTML button control in XBL, the form submit logic needs to be moved to nsHTMLInputElement::HandleDOMEvent(). Currently it resides in nsHTMLButtonControlFrame::MouseClicked().

Comment on attachment 68457 [details] [diff] [review] patch r=jkeiser if you just put a big fat comment in MouseClicked() saying this is not called (and maybe where to look for the click handler)

Comment on attachment 68457 [details] [diff] [review] patch And remove the code from image the same way.

Comment on attachment 68457 [details] [diff] [review] patch sr=attinasi Do we want to remove the same calls from the ImageControlFrame and the TextControlFrame, and if so, the implementation from the FormControlHelper? It is confusing to have some elements submitting from the frames, and some from the content element.

I had planned to go back and do some of this sort of cleanup once XBL form controls are turned on permanently, when the frame code is removed.

checked in. i will clean up this code after XBL form controls are turned on, when the old form control frames are removed.