[Webkit-unassigned] [Bug 50663] Support validation API for <object>

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=50663


Kent Tamura <tkent at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #75997|review?                     |review-
               Flag|                            |




--- Comment #3 from Kent Tamura <tkent at chromium.org>  2010-12-08 18:17:43 PST ---
(From update of attachment 75997)
View in context: https://bugs.webkit.org/attachment.cgi?id=75997&action=review

> LayoutTests/fast/forms/ValidityState-001-expected.txt:7
> +SUCCESS
> +SUCCESS
> +SUCCESS
>  SUCCESS

The test result readability is bad.
However it's ok.  You just followed the existing test.

> LayoutTests/fast/forms/checkValidity-001-expected.txt:10
> +PASS v[i].checkValidity() is true
> +PASS v[i].checkValidity() is true
> +PASS v[i].checkValidity() is true
>  PASS v[i].checkValidity() is true

ditto.

> LayoutTests/fast/forms/script-tests/setCustomValidity-existence.js:19
> +    + '</form>';
> +var controls = document.getElementsByName('victim');
> +    for (var i = 0; i < controls.length; i++)
> +        shouldBe('typeof controls[i].setCustomValidity', '"function"');

The indentation looks wrong.

The test result readability is not good.
shouldBe('document.getElementsByTagName("input")[0].setCustomValidity', '"function"');
shouldBe('document.getElementsByTagName("textarea")[0].setCustomValidity', '"function"');
...
is better.

> LayoutTests/fast/forms/willvalidate-expected.txt:51
> +Keygen element
> +FIXME: Let keygen.willValidate be false because <keygen> is barred from constraint validation.
> +FAIL document.getElementsByTagName("keygen")[0].willValidate should be false. Was true.

We should remove test cases for keygen, or should expand this bug so that it includes <keygen> fix.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.