Add MimeUtil::Parse{Audio|Video}CodecString

media/base/mime_util.h

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_BASE_MIME_UTIL_H_
 #define MEDIA_BASE_MIME_UTIL_H_
 
 #include <string>
 #include <vector>
 
+#include "media/base/audio_codecs.h"
 #include "media/base/media_export.h"
+#include "media/base/video_codecs.h"
 
 namespace media {
 
 // Check to see if a particular MIME type is in the list of
 // supported/recognized MIME types.
 MEDIA_EXPORT bool IsSupportedMediaMimeType(const std::string& mime_type);
 
 // Splits various codecs into |codecs_out|, conditionally stripping the profile
 // and level info when |strip| == true. For example, passed "aaa.b.c,dd.eee", if
 // |strip| == true |codecs_out| will contain {"aaa", "dd"}, if |strip| == false
 // |codecs_out| will contain {"aaa.b.c", "dd.eee"}.
 // See http://www.ietf.org/rfc/rfc4281.txt.
 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.
   IsNotSupported,
 
   // The given MIME type and codec combination is supported.
   IsSupported,
 
   // There's not enough information to determine if the given MIME type and
   // codec combination can be rendered or not before actually trying to play it.
@@ skipping 22 matching lines @@
 
 // Test only method that removes proprietary media types and codecs from the
 // list of supported MIME types and codecs. These types and codecs must be
 // removed to ensure consistent layout test results across all Chromium
 // variations.
 MEDIA_EXPORT void RemoveProprietaryMediaTypesAndCodecsForTests();
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_MIME_UTIL_H_