Open Bug 1336402 Opened 8 years ago Updated 2 years ago

Remove HTMLMediaElement::mozGetMetadata

Categories

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

task

Tracking

()

People

(Reporter: Ms2ger, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-needed, site-compat)

      No description provided.
Component: Audio/Video → Audio/Video: Playback
Type: defect → task
Assignee: nobody → bvandyk

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

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.

Flags: needinfo?(bvandyk)

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

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

Assignee: brycebugemail → nobody
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.