Bug 116674

Summary: [BlackBerry] Validate form data before doing direct submission.
Product: WebKit Reporter: Mike Fenton <mifenton>
Component: WebKit BlackBerryAssignee: Mike Fenton <mifenton>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Validation patch.
xan.lopez: review-
tabIndex patch
none
Updated Patch none

Description Mike Fenton 2013-05-23 07:26:13 PDT 
Two fixes:

1) Improve the submission trigger to validate the form.
2) Respect TabIndex when calculating next/prev elements.
Comment 1 Mike Fenton 2013-05-23 07:30:37 PDT 
Created attachment 202711 [details]
Validation patch.
Comment 2 Mike Fenton 2013-05-23 07:31:21 PDT 
Created attachment 202712 [details]
tabIndex patch
Comment 3 Mike Fenton 2013-05-23 08:04:32 PDT 
Comment on attachment 202712 [details]
tabIndex patch

Clearing flags, moved to new bug 116676.
Comment 4 Xan Lopez 2013-05-23 08:06:27 PDT 
Comment on attachment 202711 [details]
Validation patch.

View in context: https://bugs.webkit.org/attachment.cgi?id=202711&action=review

> Source/WebKit/blackberry/ChangeLog:3
> +        [BlackBerry] Improve input form control handling.

We should do one patch per bug. I suggest we rename this to something related to form validation, and move the other patch elsewhere.

> Source/WebKit/blackberry/WebKitSupport/InputHandler.cpp:1022
> +        formElement->submit();

Makes sense to me otherwise, seems it just moves code around to check validity before doing submit(). So please rename the title to land it!
Comment 5 Mike Fenton 2013-05-23 08:10:07 PDT 
Created attachment 202716 [details]
Updated Patch
Comment 6 Xan Lopez 2013-05-23 08:12:21 PDT 
Comment on attachment 202716 [details]
Updated Patch

r=me
Comment 7 WebKit Commit Bot 2013-05-23 08:54:56 PDT 
Comment on attachment 202716 [details]
Updated Patch

Clearing flags on attachment: 202716

Committed r150587: <http://trac.webkit.org/changeset/150587>
Comment 8 WebKit Commit Bot 2013-05-23 08:54:58 PDT 
All reviewed patches have been landed.  Closing bug.