Add support for input.minLength / textArea.minLength:
Chrome implements this, Firefox does not yet.
Created attachment 288046 [details]
Created attachment 288048 [details]
Created attachment 288054 [details]
Comment on attachment 288054 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=288054&action=review
> + // Return false for the default value or a value set by a script even if
> + // it is longer than maxLength.
Comment here mentions maxLength. That must be wrong.
> + bool tooShort(const String&, NeedsToCheckDirtyFlag) const;
Seems like this should take a StringView, not a String.
Created attachment 288062 [details]
Created attachment 288065 [details]
Comment on attachment 288065 [details]
Clearing flags on attachment: 288065
Committed r205524: <http://trac.webkit.org/changeset/205524>
All reviewed patches have been landed. Closing bug.
The "Assigning non-numeric to minlength sets minlength to 0" test in
is still failing.
*** Bug 149832 has been marked as a duplicate of this bug. ***
(In reply to comment #9)
> The "Assigning non-numeric to minlength sets minlength to 0" test in
> is still failing.
Thanks for letting me know. I'll fix this soon:
We should also treat CRLF as one character per the recent spec change if we haven't done so.