Bug 8156 – FrameMac::submitForm is busted after Vector changes

• Component: Forms
• Status: RESOLVED FIXED

[Eric Seidel (no email)]

I made some changes a while back to how form submission worked, and I busted FrameMac::submitForm.  Strangely enough, no one has seemed to notice.  However, WebKit is no longer seeing any form submission data, like it used to.

This call:

    if (!request.doPost()) {
        [_bridge loadURL:request.url().getNSURL()
                referrer:[_bridge referrer] 
                  reload:request.reload
             userGesture:true
                  target:request.frameName
         triggeringEvent:_currentEvent
                    form:_formAboutToBeSubmitted
              formValues:_formValuesAboutToBeSubmitted];

_formAboutToBeSubmitted
and
_formValuesAboutToBeSubmitted
are always nil.

Those values are now held down on Frame, and just need to be converted from WebCore types into NS types and sent up to WebKit.  It's simple to fix.  I'm filing this bug so I don't forget.

Maybe somone who knows more about that code path can comment as to why no one has noticed this being broken.  Maybe WebKit no longer needs to know about form submission data (unless it's for autofill?)

[Darin Adler]

Yes, it's primarily for auto-fill.

[Adele Peterson]

I have a patch to fix this.