1336402 - Remove HTMLMediaElement::mozGetMetadata

Status: NEW

(Core :: Audio/Video: Playback, task, P3)

Comment 1

[Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail]

We have some existing test files that are exercised via this API[0]. We could

• Remove them since we won't be able to test them anymore.
• Gtest that we still parse them correctly, so we can still store the metadata internally and have coverage for doing so.

I'd prefer the latter if it's not going to be time consuming to do.

[0] https://searchfox.org/mozilla-central/rev/d25eb00ab4e90cc0130cd18f303a04cc2a2f8409/dom/media/test/manifest.js#972

Comment 2

[Alastor Wu [:alwu]]

What about removing mozGetMetadata from the main interface, and add it as a ChromeOnly method in a partial interface? Then we can keep our tests and prevent exposing it to users.

Comment 3

[Bryce Seager van Dyk [:bryce] (he/him) - Not reading bugmail]

That's a good suggestion and is certainly a more easily executed path for now. Holding NI so I remember to do so.

Comment 4

[Chris Peterson [:cpeterson]]

Unassigning bugs assigned to Bryce because he no longer works at Mozilla.