Can you be more specific, what depth for what formats you need exactly?

PNG mostly. I need the color bits information (8, 16, 24 with or without alpha)

Created attachment 232826 [details] [review]
typefinding: add format, depth and parsed fields to PNG caps

Perhaps this would help?

Yields caps like this:

$ gst-typefind-1.0 ~/.cache/epiphany-browser/* 2>/dev/null | grep png | sed -e 's/^.*image/image/'
image/png, width=(int)423, height=(int)172, format=(string)RGB, depth=(int)24
image/png, width=(int)90, height=(int)34, format=(string)RGBP, depth=(int)8
image/png, width=(int)32, height=(int)32, format=(string)RGBA, depth=(int)32
image/png, width=(int)192, height=(int)324, format=(string)RGB, depth=(int)24
image/png, width=(int)57, height=(int)18, format=(string)RGBA, depth=(int)16
image/png, width=(int)100, height=(int)100, format=(string)RGB, depth=(int)24
image/png, width=(int)100, height=(int)100, format=(string)RGB, depth=(int)24
image/png, width=(int)32, height=(int)32, format=(string)RGBA, depth=(int)16
image/png, width=(int)175, height=(int)50, format=(string)RGBP, depth=(int)8
image/png, width=(int)64, height=(int)64, format=(string)RGB, depth=(int)24
image/png, width=(int)32, height=(int)32, format=(string)RGBA, depth=(int)16
image/png, width=(int)442, height=(int)32, format=(string)RGB, depth=(int)24
image/png, width=(int)256, height=(int)256, format=(string)RGBP, depth=(int)8
image/png, width=(int)369, height=(int)367, format=(string)RGB, depth=(int)24
image/png, width=(int)56, height=(int)37, format=(string)RGBA, depth=(int)16
image/png, width=(int)167, height=(int)924, format=(string)RGBA, depth=(int)16
image/png, width=(int)190, height=(int)80, format=(string)RGBA, depth=(int)16
image/png, width=(int)940, height=(int)373, format=(string)RGBA, depth=(int)16
image/png, width=(int)174, height=(int)174, format=(string)RGB, depth=(int)24
image/png, width=(int)7, height=(int)6, format=(string)RGBA, depth=(int)16
image/png, width=(int)16, height=(int)26, format=(string)RGB, depth=(int)24
image/png, width=(int)110, height=(int)29, format=(string)RGBA, depth=(int)16
image/png, width=(int)16, height=(int)656, format=(string)RGBA, depth=(int)16
image/png, width=(int)120, height=(int)46, format=(string)RGBA, depth=(int)16
image/png, width=(int)2500, height=(int)21, format=(string)RGBA, depth=(int)16
image/png, width=(int)132, height=(int)109, format=(string)RGBA, depth=(int)16
image/png, width=(int)21, height=(int)21, format=(string)RGBA, depth=(int)16
image/png, width=(int)300, height=(int)250, format=(string)RGBP, depth=(int)8
image/png, width=(int)32, height=(int)32, format=(string)RGBA, depth=(int)16
image/png, width=(int)9, height=(int)7, format=(string)RGBA, depth=(int)16
image/png, width=(int)11, height=(int)11, format=(string)RGBP, depth=(int)8

Question is what depth means, e.g for paletted RGB. Maybe the field should be called something else than "format", not sure.

depth seems to be redundant in combination with format. The format already contains the depth information

There can be different bpp for each format.

Ah so it's not a mapping to the GStreamer video/x-raw formats. I don't mind calling it format though, or you could call it colorspace like in JPEG2000.

See http://en.wikipedia.org/wiki/Portable_Network_Graphics#Color_depth  for the full range of possibilities.

Could make it lower-case and spell it out to differentiate from the video formats.

if that ends up in discover info, that'll help alot

I checked our profiles, it was using every bitrate anyway, so we can probably close it and I remove the restriction from our code

depths, not bitrates.

Can be closed from my pov

So you don't need the patch after all? ok

No, it wasn't used anyway.