Update is-valid flag upon resetting input elements.

Update is-valid flag upon resetting input elements.

If the value of an input element is reset back to empty, make sure we
sync its is-valid flag when doing so. If not, form control elements
marked as 'required' may end up being incorrectly labelled as valid.

R=
BUG=336818
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=165942

[sof, 2014-01-28 10:29:10]

When you next have a spare moment, please take a look.

[tkent, 2014-01-28 11:05:46]

lgtm

https://codereview.chromium.org/148463008/diff/1/LayoutTests/fast/forms/file/...
File LayoutTests/fast/forms/file/file-input-reset-validation.html (right):

https://codereview.chromium.org/148463008/diff/1/LayoutTests/fast/forms/file/...
LayoutTests/fast/forms/file/file-input-reset-validation.html:9: function
log(message)
nit: I recommend to use js-test.js.

[sof, 2014-01-28 11:38:50]

Thanks (giving it a go if the trybots still return green.)

https://codereview.chromium.org/148463008/diff/1/LayoutTests/fast/forms/file/...
File LayoutTests/fast/forms/file/file-input-reset-validation.html (right):

https://codereview.chromium.org/148463008/diff/1/LayoutTests/fast/forms/file/...
LayoutTests/fast/forms/file/file-input-reset-validation.html:9: function
log(message)
On 2014/01/28 11:05:46, tkent wrote:
> nit: I recommend to use js-test.js.

Thanks, good idea. Switched over now.

(There's an opportunity to reduce duplication and tidy up tests in this
directory; adding it to my ToDo list :) )

[commit-bot: I haz the power, 2014-01-28 12:13:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/sigbjornf@opera.com/148463008/20001

[commit-bot: I haz the power, 2014-01-28 15:25:42]

Change committed as 165942