1808599 - Perma [Tier-2] /trusted-types/trusted-types-event-handlers.tentative.html | Event handler div.ontouchstart should be blocked.

Status: RESOLVED INCOMPLETE

(Core :: DOM: Security, defect, P5)

Description

[Treeherder Bug Filer]

Filed by: nbeleuzu [at] mozilla.com
Parsed log: https://treeherder.mozilla.org/logviewer?job_id=401370129&repo=mozilla-central
Full log: https://firefox-ci-tc.services.mozilla.com/api/queue/v1/task/FNpRh7FiRgm6dacEAKqQzQ/runs/0/artifacts/public/logs/live_backing.log

---

[task 2023-01-04T19:04:02.752Z] 19:04:02     INFO - TEST-PASS | /trusted-types/trusted-types-event-handlers.tentative.html | Non-event handler div.tabIndex should not be blocked. 
[task 2023-01-04T19:04:02.752Z] 19:04:02     INFO - TEST-FAIL | /trusted-types/trusted-types-event-handlers.tentative.html | Event handler div.onerror should be blocked. - assert_throws_js: function "_ => element.setAttribute(name, "2+2")" did not throw
[task 2023-01-04T19:04:02.752Z] 19:04:02     INFO - @http://web-platform.test:8000/trusted-types/trusted-types-event-handlers.tentative.html:43:23
[task 2023-01-04T19:04:02.752Z] 19:04:02     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2591:25
[task 2023-01-04T19:04:02.752Z] 19:04:02     INFO - test@http://web-platform.test:8000/resources/testharness.js:628:30
[task 2023-01-04T19:04:02.752Z] 19:04:02     INFO - @http://web-platform.test:8000/trusted-types/trusted-types-event-handlers.tentative.html:42:9
[task 2023-01-04T19:04:02.752Z] 19:04:02     INFO - TEST-UNEXPECTED-FAIL | /trusted-types/trusted-types-event-handlers.tentative.html | Event handler div.ontouchstart should be blocked. - assert_throws_js: function "_ => element.setAttribute(name, "2+2")" did not throw
[task 2023-01-04T19:04:02.752Z] 19:04:02     INFO - @http://web-platform.test:8000/trusted-types/trusted-types-event-handlers.tentative.html:43:23
[task 2023-01-04T19:04:02.752Z] 19:04:02     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2591:25
[task 2023-01-04T19:04:02.752Z] 19:04:02     INFO - test@http://web-platform.test:8000/resources/testharness.js:628:30
[task 2023-01-04T19:04:02.752Z] 19:04:02     INFO - @http://web-platform.test:8000/trusted-types/trusted-types-event-handlers.tentative.html:42:9
[task 2023-01-04T19:04:02.753Z] 19:04:02     INFO - 
[task 2023-01-04T19:04:02.754Z] 19:04:02     INFO - TEST-UNEXPECTED-FAIL | /trusted-types/trusted-types-event-handlers.tentative.html | Event handler div.ontouchend should be blocked. - assert_throws_js: function "_ => element.setAttribute(name, "2+2")" did not throw
[task 2023-01-04T19:04:02.754Z] 19:04:02     INFO - @http://web-platform.test:8000/trusted-types/trusted-types-event-handlers.tentative.html:43:23
[task 2023-01-04T19:04:02.754Z] 19:04:02     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2591:25
[task 2023-01-04T19:04:02.754Z] 19:04:02     INFO - test@http://web-platform.test:8000/resources/testharness.js:628:30
[task 2023-01-04T19:04:02.754Z] 19:04:02     INFO - @http://web-platform.test:8000/trusted-types/trusted-types-event-handlers.tentative.html:42:9
[task 2023-01-04T19:04:02.755Z] 19:04:02     INFO - 

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1806971

Comment 2

[James Graham [:jgraham]]

I think this was fixed in Bug 1808568

Comment 3

[Intermittent Failures Robot]

11 failures in 2607 pushes (0.004 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• mozilla-central: 11

Platform and build breakdown:

• android-em-7-0-x86_64-shippable-lite-qr: 8
  • opt: 8
• android-em-7-0-x86_64-shippable-qr: 3
  • opt: 3

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1808599&startday=2023-01-02&endday=2023-01-08&tree=all

Comment 4

[Narcis Beleuzu [:NarcisB]]

It`s still failing on central

Comment 5

[Joel Maher ( :jmaher ) (UTC -8)]

odd, the change is in there and from what I see it is the correct test which would be expected: FAIL:
https://searchfox.org/mozilla-central/source/testing/web-platform/meta/trusted-types/trusted-types-event-handlers.tentative.html.ini#305

:jgraham, do you think the harness is not parsing this properly?

Comment 6

[James Graham [:jgraham]]

https://hg.mozilla.org/mozilla-central/file/tip/testing/web-platform/meta/trusted-types/trusted-types-event-handlers.tentative.html.ini suggests these changes aren't in mozilla-central. Seems like they were removed in Bug 1808950. But the current in-progress sync will re-add them: https://hg.mozilla.org/try/diff/2e399199f228a5f14d04490e8555e29067e363ed/testing/web-platform/meta/trusted-types/trusted-types-event-handlers.tentative.html.ini. So I don't think the metadata parsing is going wrong, but something odd is happening.

It looks like the test is trying to query all the properties of a div element (for (name in div.__proto__)) and it only affects the touch attributes, so my guess is for some reason we're jumping between configurations where those properties are defined and ones where they aren't.

Comment 7

[Intermittent Failures Robot]

4 failures in 3530 pushes (0.001 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• mozilla-central: 4

Platform and build breakdown:

• android-em-7-0-x86_64-shippable-qr: 2
  • opt: 2
• android-em-7-0-x86_64-shippable-lite-qr: 2
  • opt: 2

For more details, see:
https://treeherder.mozilla.org/intermittent-failures/bugdetails?bug=1808599&startday=2023-01-09&endday=2023-01-15&tree=all

Comment 8

[BugBot [:suhaib / :marco/ :calixte]]

Set release status flags based on info from the regressing bug 1806971

Comment 9

[BugBot [:suhaib / :marco/ :calixte]]

https://wiki.mozilla.org/Bug_Triage#Intermittent_Test_Failure_Cleanup
For more information, please visit auto_nag documentation.