Summary: | REGRESSION(r58520): Implicit submission on forms with button type="submit" no longer works | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dimitri Glazkov (Google) <dglazkov> | ||||
Component: | New Bugs | Assignee: | 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
Dimitri Glazkov (Google)
2010-05-11 09:28:37 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. Created attachment 55720 [details]
Patch
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
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. Committed r59173: <http://trac.webkit.org/changeset/59173> |