Check willValidate() for :in-range and :out-of-range.

Check willValidate() for :in-range and :out-of-range.

:in-range and :out-of-range should only match elements which are
candidates for constraint validation[1], and they are not when they are
disabled or read-only.

https://html.spec.whatwg.org/multipage/scripting.html#selector-in-range

R=tkent@chromium.org
BUG=602568
Committed: https://crrev.com/cb289485c838f65f92a48813704b7671388c6c62
Cr-Commit-Position: refs/heads/master@{#387258}

[rune, 2016-04-13 20:08:18]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[rune, 2016-04-13 20:08:26]

ptal

[commit-bot: I haz the power, 2016-04-13 20:08:35]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1890633002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1890633002/1

[commit-bot: I haz the power, 2016-04-13 20:20:59]

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

[commit-bot: I haz the power, 2016-04-13 20:21:01]

Dry run: Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)

[rune, 2016-04-13 21:37:56]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-04-13 21:38:23]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1890633002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1890633002/20001

[commit-bot: I haz the power, 2016-04-13 22:50:13]

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

[commit-bot: I haz the power, 2016-04-13 22:50:14]

Dry run: This issue passed the CQ dry run.

[tkent, 2016-04-13 22:52:54]

lgtm.

I think we don't need to add new tests to fast/css because
imported/web-platform-tests/html/semantics/selectors/pseudo-classes/inrange-outofrange.html
covers these cases.
If you'd like to add testcases to fast/css, I prefer adding them to existing
fast/css/pseudo-in-range.html and pseudo-out-of-range.html to adding new test
file.

[rune, 2016-04-14 06:36:20]

On 2016/04/13 22:52:54, tkent wrote:
> lgtm.
> 
> I think we don't need to add new tests to fast/css because
>
imported/web-platform-tests/html/semantics/selectors/pseudo-classes/inrange-outofrange.html
> covers these cases.

Removed.

[rune, 2016-04-14 06:36:30]

The CQ bit was checked by rune@opera.com

[rune, 2016-04-14 06:36:31]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/1890633002/#ps40001
(title: "Removed extra tests")

[commit-bot: I haz the power, 2016-04-14 06:36:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1890633002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1890633002/40001

[commit-bot: I haz the power, 2016-04-14 07:46:42]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-04-14 07:49:32]

Description was changed from

==========
Check willValidate() for :in-range and :out-of-range.

:in-range and :out-of-range should only match elements which are
candidates for constraint validation[1], and they are not when they are
disabled or read-only.

https://html.spec.whatwg.org/multipage/scripting.html#selector-in-range

R=tkent@chromium.org
BUG=602568
==========

to

==========
Check willValidate() for :in-range and :out-of-range.

:in-range and :out-of-range should only match elements which are
candidates for constraint validation[1], and they are not when they are
disabled or read-only.

https://html.spec.whatwg.org/multipage/scripting.html#selector-in-range

R=tkent@chromium.org
BUG=602568

Committed: https://crrev.com/cb289485c838f65f92a48813704b7671388c6c62
Cr-Commit-Position: refs/heads/master@{#387258}
==========

[commit-bot: I haz the power, 2016-04-14 07:49:33]

Patchset 3 (id:??) landed as
https://crrev.com/cb289485c838f65f92a48813704b7671388c6c62
Cr-Commit-Position: refs/heads/master@{#387258}