Add MimeUtil::Parse{Audio|Video}CodecString

media/base/mime_util.h

Index: media/base/mime_util.h
diff --git a/media/base/mime_util.h b/media/base/mime_util.h
index d7a39f41a959dc2045d2addab12cbb2045b53be2..b25391b97c35f141919a5c8e4734dc8e52366a99 100644
--- a/media/base/mime_util.h
+++ b/media/base/mime_util.h
@@ -8,7 +8,9 @@
 #include <string>
 #include <vector>
 
+#include "media/base/audio_codecs.h"
 #include "media/base/media_export.h"
+#include "media/base/video_codecs.h"
 
 namespace media {
 
@@ -25,6 +27,19 @@ MEDIA_EXPORT void SplitCodecsToVector(const std::string& codecs,
                                       std::vector<std::string>* codecs_out,
                                       bool strip);
 
+MEDIA_EXPORT bool ParseVideoCodecString(const std::string& mime_type,
+                                        const std::string& codec_id,
+                                        bool* out_is_ambiguous,
+                                        VideoCodec* out_codec,
+                                        VideoCodecProfile* out_profile,
+                                        uint8_t* out_level,
+                                        VideoColorSpace* out_colorspace);
+
+MEDIA_EXPORT bool ParseAudioCodecString(const std::string& mime_type,
+                                        const std::string& codec_id,
+                                        bool* out_is_ambiguous,
+                                        AudioCodec* out_codec);

[mlamouri (slow - plz ping), 2017/05/11 09:59:07]

Here and for the internal header too, it would be good to give an explanation of
what the methods are doing and what to expect. Especially given tha they are
taking out params and also have a return values. As a caller, should I expect
`out_level` to be set if `ParseVideoCodecString` returns false for example?

+
 // Indicates that the MIME type and (possible codec string) are supported.
 enum SupportsType {
   // The given MIME type and codec combination is not supported.