FIELDSET element should not check inner controls in checkValidity().

FIELDSET element should not check inner controls in checkValidity().

fieldset.checkValidity() referred to inner control validity state by accident.
The common implmentatin of checkValidity() just calls isValidElement(), and
FIELDSET's isValidElement() checks inner controls validity for :valid/:invalid.

This CL adds willValidate check to the common checkValidity(). Also, it exposed
an issue described in HTMLFormControlElement::setNeedsWillValidateCheck()
because we used to clear m_validityIsDirty in checkValidity().

BUG=633794
Committed: https://crrev.com/52ccdca1f50ae313029eb6e996a3ef4288b3f638
Cr-Commit-Position: refs/heads/master@{#409500}

[tkent, 2016-08-03 02:34:22]

The CQ bit was checked by tkent@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-03 02:34:43]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[tkent, 2016-08-03 03:15:07]

Description was changed from

==========
FIELDSET element should not check inner controls in checkValidity().

fieldset.checkValidity() referred to inner control validity state by accident.
The common implmentatin of checkValidity() just calls isValidElement(), and
FIELDSET's isValidElement() checks inner controls validitiy for :valid/:invalid.

Thic CL adds willValidate check to the common checkValidity(). Also, it exposed
an issue described in HTMLFormControlElement::setNeedsWillValidateCheck()
because we used to clear m_alidityIsDirty in checkValidity().

BUG=633794
==========

to

==========
FIELDSET element should not check inner controls in checkValidity().

fieldset.checkValidity() referred to inner control validity state by accident.
The common implmentatin of checkValidity() just calls isValidElement(), and
FIELDSET's isValidElement() checks inner controls validity for :valid/:invalid.

This CL adds willValidate check to the common checkValidity(). Also, it exposed
an issue described in HTMLFormControlElement::setNeedsWillValidateCheck()
because we used to clear m_validityIsDirty in checkValidity().

BUG=633794
==========

[tkent, 2016-08-03 05:05:05]

tkent@chromium.org changed reviewers:
+ keishi@chromium.org

[tkent, 2016-08-03 05:05:06]

Keishi, would you review this please?

[commit-bot: I haz the power, 2016-08-03 06:00:00]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-03 06:00:00]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[keishi, 2016-08-03 09:44:02]

LGTM

[tkent, 2016-08-03 11:56:37]

The CQ bit was checked by tkent@chromium.org

[commit-bot: I haz the power, 2016-08-03 11:56:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-03 13:23:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-03 13:26:02]

Description was changed from

==========
FIELDSET element should not check inner controls in checkValidity().

fieldset.checkValidity() referred to inner control validity state by accident.
The common implmentatin of checkValidity() just calls isValidElement(), and
FIELDSET's isValidElement() checks inner controls validity for :valid/:invalid.

This CL adds willValidate check to the common checkValidity(). Also, it exposed
an issue described in HTMLFormControlElement::setNeedsWillValidateCheck()
because we used to clear m_validityIsDirty in checkValidity().

BUG=633794
==========

to

==========
FIELDSET element should not check inner controls in checkValidity().

fieldset.checkValidity() referred to inner control validity state by accident.
The common implmentatin of checkValidity() just calls isValidElement(), and
FIELDSET's isValidElement() checks inner controls validity for :valid/:invalid.

This CL adds willValidate check to the common checkValidity(). Also, it exposed
an issue described in HTMLFormControlElement::setNeedsWillValidateCheck()
because we used to clear m_validityIsDirty in checkValidity().

BUG=633794

Committed: https://crrev.com/52ccdca1f50ae313029eb6e996a3ef4288b3f638
Cr-Commit-Position: refs/heads/master@{#409500}
==========

[commit-bot: I haz the power, 2016-08-03 13:26:03]

Patchset 1 (id:??) landed as
https://crrev.com/52ccdca1f50ae313029eb6e996a3ef4288b3f638
Cr-Commit-Position: refs/heads/master@{#409500}