Bug 51289 – Initial state of checkValidity() and :invalid are incorrect with <select required>

• Status: RESOLVED
• Resolution: FIXED
• Component: Forms

[Kent Tamura]

Created attachment 76939 [details]
Reproducing HTML

Open the attached HTML.
It shows "valueMissing=false; valid=true; checkValidity()=false" though it should show "valueMissing=false; valid=true; checkValidity()=true".
It looks red (:invalid applied) though it should not look red.

http://code.google.com/p/chromium/issues/detail?id=67290

[Dai Mikurube]

Created attachment 77183 [details]
Patch

[Kent Tamura]

Comment on attachment 77183 [details]
Patch

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

> LayoutTests/ChangeLog:7
> +        https://bugs.webkit.org/show_bug.cgi?id=51289
> +        Initial state of checkValidity() and :invalid are incorrect with <select required>
> +

nit: We usually put a summary, then a bug URL.

[Kent Tamura]

Comment on attachment 77183 [details]
Patch

Anyway, ok.

[WebKit Commit Bot]

Comment on attachment 77183 [details]
Patch

Clearing flags on attachment: 77183

Committed r74470: <http://trac.webkit.org/changeset/74470>

[WebKit Commit Bot]

Comment on attachment 77183 [details]
Patch

Rejecting attachment 77183 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-3', 'land-attachment', '--force-clean', '--ignore-builders', '--non-interactive', '--parent-command=commit-queue', 77183]" exit_code: 1
Last 500 characters of output:
ebkit-commit-queue/Tools/Scripts/webkitpy/thirdparty/autoinstalled/mechanize/_html.py", line 546, in __getattr__
    self.forms()
  File "/mnt/git/webkit-commit-queue/Tools/Scripts/webkitpy/thirdparty/autoinstalled/mechanize/_html.py", line 559, in forms
    self._forms_factory.forms())
  File "/mnt/git/webkit-commit-queue/Tools/Scripts/webkitpy/thirdparty/autoinstalled/mechanize/_html.py", line 228, in forms
    raise ParseError(exc)
webkitpy.thirdparty.autoinstalled.mechanize._html.ParseError

Full output: http://queues.webkit.org/results/7250096

[Dai Mikurube]

(In reply to comment #5)
Does anyone know the current status?
The patch looks committed into the tree, and it can be built correctly despite this error in applying the patch.