After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 795708 - gsttaglist: Add new tag GST_TAG_RFC6381_CODECS
gsttaglist: Add new tag GST_TAG_RFC6381_CODECS
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
1.14.0
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2018-05-01 03:17 UTC by Yeongjin Jeong
Modified: 2018-11-03 12:45 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Suggests adding the following new tags: GST_TAG_RFC6381_CODECS. (2.13 KB, patch)
2018-05-01 03:17 UTC, Yeongjin Jeong
none Details | Review
hlsdemux: push rfc6381-codecs tag (2.59 KB, patch)
2018-05-01 03:19 UTC, Yeongjin Jeong
none Details | Review
dashdemux: Remove white space and run gst-indent (1.68 KB, patch)
2018-05-01 03:22 UTC, Yeongjin Jeong
none Details | Review
dashdemux: push rfc6381-codecs tag (2.81 KB, patch)
2018-05-01 03:22 UTC, Yeongjin Jeong
none Details | Review

Description Yeongjin Jeong 2018-05-01 03:17:52 UTC
Created attachment 371566 [details] [review]
Suggests adding the following new tags: GST_TAG_RFC6381_CODECS.

Suggests adding the following new tags: GST_TAG_RFC6381_CODECS.

This is useful for information which creates Live streaming protocol metadata such as M3U8 master playlist or MPD manifest.

Becase, some media player framework such as MSE (Media Source Extensions) need more information 'rfc6381-codecs' to test support for in the current browser.
=> https://developer.mozilla.org/en-US/docs/Web/API/MediaSource/isTypeSupported

Also I uploaded patches that push rfc6381-codecs tag in hlsdemuxer and dashdemuxer.
Comment 1 Yeongjin Jeong 2018-05-01 03:19:43 UTC
Created attachment 371567 [details] [review]
hlsdemux: push rfc6381-codecs tag

RFC6381-codecs tag is retrieved from the EXT-X-STREAM-INF codecs property at master playlist.
Comment 2 Yeongjin Jeong 2018-05-01 03:22:04 UTC
Created attachment 371568 [details] [review]
dashdemux: Remove white space and run gst-indent

dashdemux: Remove white space and run gst-indent
Comment 3 Yeongjin Jeong 2018-05-01 03:22:27 UTC
Created attachment 371569 [details] [review]
dashdemux: push rfc6381-codecs tag

RFC6381-codecs tag is retrieved from the RepresentationBase codecs property.
Comment 4 Sebastian Dröge (slomo) 2018-05-01 06:11:47 UTC
This is only a single codec per tag, not codec*s*, or not?
Comment 5 Sebastian Dröge (slomo) 2018-05-01 06:14:51 UTC
It might also be good to have some kind of caps<->rfc6381 conversion in pbutils
Comment 6 Yeongjin Jeong 2018-05-01 10:04:01 UTC
(In reply to Sebastian Dröge (slomo) from comment #4)
> This is only a single codec per tag, not codec*s*, or not?

It is defined as a parameter called 'codecs' in RFC6381. (https://tools.ietf.org/html/rfc6381#page-5)

Also, as an example I missed, codecs can contain more than one codec.
e.g. "mp4a.40.2, avc1.4d001e" (AAC-LC, H.264 Main Profile level 3.0)
Comment 7 Yeongjin Jeong 2018-05-01 10:22:23 UTC
(In reply to Sebastian Dröge (slomo) from comment #5)
> It might also be good to have some kind of caps<->rfc6381 conversion in
> pbutils

I agree that it would be nice.

However, the specification of RFC6381 is broad to support all CAPS to RFC6381 conversion in this thread.
So I would like to suggest how about implementing it in the next step, what do you think about?
Comment 8 Sebastian Dröge (slomo) 2018-05-02 05:30:03 UTC
(In reply to Yeongjin Jeong from comment #7)

> So I would like to suggest how about implementing it in the next step, what
> do you think about?

Yes I agree, one step at a time. But can you open a enhancement bug about the caps part?
Comment 9 Yeongjin Jeong 2018-05-02 12:38:11 UTC
(In reply to Sebastian Dröge (slomo) from comment #8)
> (In reply to Yeongjin Jeong from comment #7)
> 
> > So I would like to suggest how about implementing it in the next step, what
> > do you think about?
> 
> Yes I agree, one step at a time. But can you open a enhancement bug about
> the caps part?

Yes, of course. That suggestion registered at Bug 795752.
[bug #795752](https://bugzilla.gnome.org/show_bug.cgi?id=795752)
Comment 10 GStreamer system administrator 2018-11-03 12:45:55 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/gstreamer/gstreamer/issues/287.