Use mime type helper to test ffmpeg extractor and add
new mime type
CCBUG: 399650
bruns |
Use mime type helper to test ffmpeg extractor and add
new mime type
CCBUG: 399650
No Linters Available |
No Unit Test Coverage |
Buildable 10262 | |
Build 10280: arc lint + arc unit |
src/extractors/ffmpegextractor.cpp | ||
---|---|---|
64 | This is quite specific to the codec. ExtractorCollection should handle this case I think (mime type of file is more specific than in the mime type list), but I added it nevertheless. |
autotests/ffmpegextractortest.cpp | ||
---|---|---|
27 | remove comment | |
src/extractors/ffmpegextractor.cpp | ||
64 | I am somewhat undecided on this one: The ffmpeg extractor leaves the handling of different types completely to libav (contrary to e.g. taglib, where each type is handled explicitly). So just defaulting to video/ogg already covers theora, and many more. I think it is better to test this in e.g. the extractorcollectiontest, and remove the QVERIFY for the mimetypes().contains(). |