[MediaCapabilities][GStreamer] Check audio codecs support isConfigurationSupported()

https://bugs.webkit.org/show_bug.cgi?id=310195

Reviewed by NOBODY (OOPS!).

Currently, isConfigurationSupported() only validates video codecs
against the registered decoder/encoder map. Audio codecs from the audio
configuration were never checked, so unsupported audio codecs could
incorrectly be reported as supported.

See: WebPlatformForEmbedded/WPEWebKit#1640

Fix this by collecting codec strings from both the video and audio
ContentType objects into a single vector and running the software codec
support checks once on the combined list at the end of the function.

Audio hardware accelerated decoding doesn't save much CPU power compared
to video accelerated decoding, and we don't want to reject a video+audio
codec combination just because the audio part isn't accelerated. That's
why the hardware accelerated decoding support checks are only run on the
video codecs.

Co-authored by: Andrzej Surdej (https://github.com/asurdej-comcast)

* Source/WebCore/platform/graphics/gstreamer/GStreamerRegistryScanner.cpp:
(WebCore::GStreamerRegistryScanner::isConfigurationSupported const): Collect audio codecs and run the codec checks.

Author: eocanha

Source/WebCore/platform/graphics/gstreamer/GStreamerRegistryScanner.cpp

@@ -1030,11 +1030,13 @@ ASCIILiteral GStreamerRegistryScanner::configurationNameForLogging(Configuration
 
 GStreamerRegistryScanner::RegistryLookupResult GStreamerRegistryScanner::isConfigurationSupported(Configuration configuration, const PlatformMediaConfiguration& mediaConfiguration) const
 {
-    bool isUsingHardware = false;
 #ifndef GST_DISABLE_GST_DEBUG
     ASCIILiteral configLogString = configurationNameForLogging(configuration);
 #endif
 
+    Vector<String> allCodecs;
+    Vector<String> videoCodecs;
+
     if (mediaConfiguration.video) {
         auto& videoConfiguration = mediaConfiguration.video.value();
 #ifndef GST_DISABLE_GST_DEBUG
@@ -1064,12 +1066,8 @@ GStreamerRegistryScanner::RegistryLookupResult GStreamerRegistryScanner::isConfi
         if (!isContainerTypeSupported(configuration, contentType.containerType()))
             return { false, false, nullptr };
 
-        auto codecs = contentType.codecs();
-        if (!codecs.isEmpty()) {
-            if (!areAllCodecsSupported(configuration, codecs, false))
-                return { false, false, nullptr };
-            isUsingHardware = areAllCodecsSupported(configuration, codecs, true);
-        }
+        allCodecs.appendVector(contentType.codecs());
+        videoCodecs.appendVector(contentType.codecs());
     }
 
     if (mediaConfiguration.audio) {
@@ -1082,6 +1080,15 @@ GStreamerRegistryScanner::RegistryLookupResult GStreamerRegistryScanner::isConfi
         auto contentType = ContentType(audioConfiguration.contentType);
         if (!isContainerTypeSupported(configuration, contentType.containerType()))
             return { false, false, nullptr };
+
+        allCodecs.appendVector(contentType.codecs());
+    }
+
+    bool isUsingHardware = false;
+    if (!allCodecs.isEmpty()) {
+        if (!areAllCodecsSupported(configuration, allCodecs, false))
+            return { false, false, nullptr };
+        isUsingHardware = areAllCodecsSupported(configuration, videoCodecs, true);
     }
 
     return { true, isUsingHardware, nullptr };