1950601 - Improve trusted-types/trusted-types-event-handlers.html

Status: RESOLVED FIXED

(Core :: DOM: Security, task)

Description

[Frédéric Wang Nélar]

https://searchfox.org/mozilla-central/rev/fa11d06fd84c487cb0a135c08af796daa6a56e44/testing/web-platform/meta/trusted-types/trusted-types-event-handlers.html.ini enables dom.event.dragexit.enabled and some tests fail on non-Android platforms. This test is similar to set-event-handlers-content-attributes.tentative.html (probably even a subtest) so probably there is a way to avoid this kind of issues like doing a check like isAttributeImplemented.

Comment 1

[Frédéric Wang Nélar]

It seems the ini file is not necessary anymore. I cannot reproduce the issue locally (the failing tests are not run) or on try:

https://treeherder.mozilla.org/jobs?repo=try&revision=6e2adb36d555aa47c096d61563ee68b3e669da79

I'm not sure what changed, though.

Comment 2

[Frédéric Wang Nélar]

Attachment: Bug 1950601 Remove trusted-types/trusted-types-event-handlers.ini. r=smaug

The test only handles attributes that belong to the div's protototype.
If a platform does not support these attributes, corresponding tests
are not executed can so cannot fail.

Comment 3

[Frédéric Wang Nélar]

https://treeherder.mozilla.org/jobs?repo=try&revision=6e2adb36d555aa47c096d61563ee68b3e669da79

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/66fe8199bef4
Remove trusted-types/trusted-types-event-handlers.ini. r=smaug

Comment 5

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/66fe8199bef4