1820474 - Write test that mixed-content-auto-upgrading does not apply to `<track>` elements

Status: NEW

(Core :: DOM: Security, task, P3)

Description

[Christoph Kerschbaumer [:ckerschb]]

Internally we match track to type media, though the spec for mixed content auto upgrading explicitly stated that we should only upgrade:

• image,
• audio
• video.

We should write an automated test to ensure track is not subject to mixed-content-auto-upgrading.

Comment 1

[Abhishek[: unavailable till 11 May]]

Sir, I want to give a try to this bug,

Comment 2

[Frederik Braun [:freddy]]

Great. If you want to tackle this bug, please follow the guide at https://wiki.mozilla.org/Good_first_bug to get a working build environment and make sure you can run the test file dom/security/test/mixedcontentblocker/test_main.html#35 using ./mach test <path to test>.

The first steps is to make a change to the test that adds a case for a <track> element.

Feel free to reach out to me as @fbraun:mozilla.org on Matrix (Instructions at https://wiki.mozilla.org/Matrix) for further questions.

Comment 3

[Abhishek[: unavailable till 11 May]]

Ok, Thanks, I will give my best

Comment 4

[Abhishek[: unavailable till 11 May]]

Sir, I am trying to submit a patch but getting the error Freddy is not a valid reviewer name, and I also looked for the same on Mozilla but not able to find you. Please have a look at this - https://paste.mozilla.org/tG8prCS1

Comment 5

[Frederik Braun [:freddy]]

That's odd. You can go without a reviewer name (though maybe "freddyb" could work).

Comment 6

[Abhishek[: unavailable till 11 May]]

Attachment: Bug 1820474 - Write test and ensure mixed-content-auto-upgrading does not apply to track elements r=freddyb

Comment 7

[Jonathan Kingston [:jkt] he/him]

Why should track be excluded, see: https://bugzilla.mozilla.org/show_bug.cgi?id=1439463

Comment 8

[Frederik Braun [:freddy]]

To clarify, we do not upgrade <track> since https://bugzilla.mozilla.org/show_bug.cgi?id=1819898. This is just about the missing test case. I think the bug mentioned above can be closed?

Comment 9

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

This good-first-bug hasn't had any activity for 2 months, it is automatically unassigned.
For more information, please visit BugBot documentation.