Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Guidance for user-defined VideoFrameMetadata entries #849

Open
chrisn opened this issue Nov 21, 2024 · 6 comments
Open

Guidance for user-defined VideoFrameMetadata entries #849

chrisn opened this issue Nov 21, 2024 · 6 comments
Assignees
Labels
agenda Add to Media WG call agenda

Comments

@chrisn
Copy link
Member

chrisn commented Nov 21, 2024

Web developers could use the VideoFrameMetadata dictionary to store their own custom metadata. Should we provide guidance in the spec or the registry, e.g., on naming convention, so developers avoid conflicts with platform-defined entries?

@chrisn
Copy link
Member Author

chrisn commented Nov 21, 2024

Metadata is copied both when passed to the VideoFrame constructor and when returned from metadata() so applications can use any custom dictionary entries without affecting the browser's no behaviour, even if they use the same names. An application might need to rename its own metadata entries if a new registry entry is defined with a name they're using and they want to use both. So could be worth giving developer advice to use something like an underscore prefix for their own metadata values?

@Djuffin
Copy link
Contributor

Djuffin commented Nov 21, 2024

Frankly I'm not sure that this the intended use for VF metadata. I would only copy metadata members known to the UA.

dictionary VideoFrameMetadata {
  // Possible members are recorded in the VideoFrame Metadata Registry.
};

@Djuffin
Copy link
Contributor

Djuffin commented Nov 21, 2024

cc @aboba @padenot for opinion

@aboba
Copy link
Collaborator

aboba commented Nov 21, 2024

The WebCodecs specification indicates that metadata dictionary entries are epected to be recorded in the VideoFrame Metadata Registry. But do we really expect the UA to only support dictionary members recorded in the Registry? This seems like a maintenance (and interop) headache. Supporting all dictionary members (including custom ones) seems simpler.

@padenot
Copy link
Collaborator

padenot commented Nov 22, 2024

I seem to recall professionals in the industry requesting metadata pass-through. Or was it @youennf ?

It's harmless enough to simply forward them.

@aboba aboba added the agenda Add to Media WG call agenda label Dec 4, 2024
@Djuffin
Copy link
Contributor

Djuffin commented Dec 10, 2024

Media WG: let's have a note in the metadata registry along the lines

This registry will never have entries starting with `some prefix`

I'll check if the existing web standards have precedence for such prefixes

@Djuffin Djuffin self-assigned this Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agenda Add to Media WG call agenda
Projects
None yet
Development

No branches or pull requests

4 participants