Bug 38913

Summary: REGRESSION(r58520): Implicit submission on forms with button type="submit" no longer works
Product: WebKit Reporter: Dimitri Glazkov (Google) <dglazkov>
Component: New BugsAssignee: Dimitri Glazkov (Google) <dglazkov>
Status: RESOLVED FIXED    
Severity: Normal CC: ap, darin
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 39021    
Attachments:
Description Flags
Patch darin: review+

Description Dimitri Glazkov (Google) 2010-05-11 09:28:37 PDT
REGRESSION(r58520): Implicit submission on forms with button type="submit" no longer works
Comment 1 Dimitri Glazkov (Google) 2010-05-11 10:31:25 PDT
Turns out my super-duper test had incomplete coverage. 

http://www.shacknews.com/search.x?terms=abcdef&type=comments no longer implicitly submits.

Patch coming up.
Comment 2 Dimitri Glazkov (Google) 2010-05-11 10:49:09 PDT
Created attachment 55720 [details]
Patch
Comment 3 Darin Adler 2010-05-11 10:55:57 PDT
Comment on attachment 55720 [details]
Patch

I would have preferred to see the bug fix separate from the cleanup. Clearly all that was needed to fix the bug was to check isSuccessfulSubmitButton even when the element is not an <input>. And the renaming and refactoring obscures that change.

The final resulting code looks great.

r=me
Comment 4 Alexey Proskuryakov 2010-05-11 11:05:34 PDT
<rdar://problem/7969230>
Comment 5 Dimitri Glazkov (Google) 2010-05-11 11:09:42 PDT
I apologize, I started with a fix and then it was looking uglier and uglier. I have another patch that makes it even less ugly, but I'll land this first, then lick it some more.
Comment 6 Dimitri Glazkov (Google) 2010-05-11 15:01:37 PDT
Committed r59173: <http://trac.webkit.org/changeset/59173>