Correct elementMatches filter for datalist.options

Correct elementMatches filter for datalist.options

The filter should match all option elements [1], even if they are empty
or disabled. This patch fixes the filter and ensures that the tests in
[2] pass.

All callers have been updated to filter on empty and disabled values,
except for the call from V8HTMLDataListElement.

The `fast/forms/datalist/datalist.html` test was removed as the correct
behavior is checked by
`wpt/html/semantics/forms/the-datalist-element/datalistoptions.html`.

[1] - https://html.spec.whatwg.org/multipage/forms.html#dom-datalist-options
[2] - http://w3c-test.org/html/semantics/forms/the-datalist-element/datalistoptions.html

BUG=687073
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2746823002
Cr-Commit-Position: refs/heads/master@{#456586}
Committed: https://chromium.googlesource.com/chromium/src/+/d08576a42a3c298ce3b8a39f92f58440446e303d

[MartinRogalla, 2017-03-13 18:34:21]

Description was changed from

==========
Correct elementMatches filter for datalist.options

The filter should match all option elements [1], even if they are empty
or disabled. This patch fixes the filter and ensures that the tests in
[2] pass.

[1] - https://html.spec.whatwg.org/multipage/forms.html#dom-datalist-options
[2] -
http://w3c-test.org/html/semantics/forms/the-datalist-element/datalistoptions...

BUG=687073
==========

to

==========
Correct elementMatches filter for datalist.options

The filter should match all option elements [1], even if they are empty
or disabled. This patch fixes the filter and ensures that the tests in
[2] pass.

[1] - https://html.spec.whatwg.org/multipage/forms.html#dom-datalist-options
[2] -
http://w3c-test.org/html/semantics/forms/the-datalist-element/datalistoptions...

BUG=687073
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[MartinRogalla, 2017-03-13 18:37:36]

Description was changed from

==========
Correct elementMatches filter for datalist.options

The filter should match all option elements [1], even if they are empty
or disabled. This patch fixes the filter and ensures that the tests in
[2] pass.

[1] - https://html.spec.whatwg.org/multipage/forms.html#dom-datalist-options
[2] -
http://w3c-test.org/html/semantics/forms/the-datalist-element/datalistoptions...

BUG=687073
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Correct elementMatches filter for datalist.options

The filter should match all option elements [1], even if they are empty
or disabled. This patch fixes the filter and ensures that the tests in
[2] pass.

All callers have been updated to filter on empty and disabled values,
except for the call from V8HTMLDataListElement.

The `fast/forms/datalist/datalist.html` test was removed as the correct
behavior is checked by
`wpt/html/semantics/forms/the-datalist-element/datalistoptions.html`.

[1] - https://html.spec.whatwg.org/multipage/forms.html#dom-datalist-options
[2] -
http://w3c-test.org/html/semantics/forms/the-datalist-element/datalistoptions...

BUG=687073
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[MartinRogalla, 2017-03-13 19:08:22]

martin@martinrogalla.com changed reviewers:
+ tkent@chromium.org

[tkent, 2017-03-13 20:06:02]

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

[commit-bot: I haz the power, 2017-03-13 20:06:35]

Dry run: 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, 2017-03-13 21:32:25]

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

[commit-bot: I haz the power, 2017-03-13 21:32:26]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[MartinRogalla, 2017-03-13 22:35:33]

On 2017/03/13 19:08:23, MartinRogalla wrote:

The failure is not due to the patch, correct?

[tkent, 2017-03-13 23:49:15]

The CQ bit was checked by tkent@chromium.org

[tkent, 2017-03-13 23:49:15]

lgtm

[commit-bot: I haz the power, 2017-03-13 23:49:42]

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, 2017-03-14 02:35:56]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1489448955481350,
"parent_rev": "f1c3dbc05ae362562665108771589d20b1d499ac", "commit_rev":
"d08576a42a3c298ce3b8a39f92f58440446e303d"}

[commit-bot: I haz the power, 2017-03-14 02:36:40]

Description was changed from

==========
Correct elementMatches filter for datalist.options

The filter should match all option elements [1], even if they are empty
or disabled. This patch fixes the filter and ensures that the tests in
[2] pass.

All callers have been updated to filter on empty and disabled values,
except for the call from V8HTMLDataListElement.

The `fast/forms/datalist/datalist.html` test was removed as the correct
behavior is checked by
`wpt/html/semantics/forms/the-datalist-element/datalistoptions.html`.

[1] - https://html.spec.whatwg.org/multipage/forms.html#dom-datalist-options
[2] -
http://w3c-test.org/html/semantics/forms/the-datalist-element/datalistoptions...

BUG=687073
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Correct elementMatches filter for datalist.options

The filter should match all option elements [1], even if they are empty
or disabled. This patch fixes the filter and ensures that the tests in
[2] pass.

All callers have been updated to filter on empty and disabled values,
except for the call from V8HTMLDataListElement.

The `fast/forms/datalist/datalist.html` test was removed as the correct
behavior is checked by
`wpt/html/semantics/forms/the-datalist-element/datalistoptions.html`.

[1] - https://html.spec.whatwg.org/multipage/forms.html#dom-datalist-options
[2] -
http://w3c-test.org/html/semantics/forms/the-datalist-element/datalistoptions...

BUG=687073
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2746823002
Cr-Commit-Position: refs/heads/master@{#456586}
Committed:
https://chromium.googlesource.com/chromium/src/+/d08576a42a3c298ce3b8a39f92f5...
==========

[commit-bot: I haz the power, 2017-03-14 02:36:41]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/d08576a42a3c298ce3b8a39f92f5...