New interface (AudioDecoderFactory), with an implementation

webrtc/modules/audio_coding/audio_coding.gypi

 # Copyright (c) 2015 The WebRTC project authors. All Rights Reserved.
 #
 # Use of this source code is governed by a BSD-style license
 # that can be found in the LICENSE file in the root of the source
 # tree. An additional intellectual property rights grant can be found
 # in the file PATENTS.  All contributing project authors may
 # be found in the AUTHORS file in the root of the source tree.
 
 {
   'includes': [
@@ skipping 51 matching lines @@
     'audio_coding_dependencies': [
       '<@(audio_codec_dependencies)',
       '<(webrtc_root)/common.gyp:webrtc_common',
       '<(webrtc_root)/common_audio/common_audio.gyp:common_audio',
       '<(webrtc_root)/system_wrappers/system_wrappers.gyp:system_wrappers',
     ],
     'audio_coding_defines': '<(audio_codec_defines)',
   },
   'targets': [
     {
+      'target_name': 'audio_codec_factory_interface',

[hlundin-webrtc, 2016/04/27 14:35:56]

I assume you are intending to extend this target (and the one below) with
encoder capabilities too. Otherwise, the naming is off.

[kwiberg-webrtc, 2016/04/28 12:35:51]

Yes, that was my plan. But perhaps it's better to name it for the current
contents, and change the name later.

+      'type': 'static_library',
+      'dependencies': [
+        '<(webrtc_root)/common.gyp:webrtc_common',
+      ],
+      'include_dirs': [
+        '<(webrtc_root)',
+      ],
+      'direct_dependent_settings': {
+        'include_dirs': [
+          '<(webrtc_root)',
+        ],
+      },
+      'sources': [
+        'codecs/audio_decoder_factory.h',
+        'codecs/audio_format.cc',
+        'codecs/audio_format.h',
+      ],
+    },
+    {
+      'target_name': 'builtin_audio_codec_factory',

[the sun, 2016/04/27 20:34:14]

_factories?

[kwiberg-webrtc, 2016/04/28 12:35:51]

I'll change codec to decoder instead; see previous comment.

+      'type': 'static_library',
+      'defines': [
+        '<@(audio_codec_defines)',
+      ],
+      'dependencies': [
+        '<(webrtc_root)/common.gyp:webrtc_common',
+        '<@(audio_codec_dependencies)',
+        'audio_codec_factory_interface',
+      ],
+      'include_dirs': [
+        '<(webrtc_root)',
+      ],
+      'direct_dependent_settings': {
+        'include_dirs': [
+          '<(webrtc_root)',
+        ],
+      },
+      'sources': [
+        'codecs/builtin_audio_decoder_factory.cc',
+        'codecs/builtin_audio_decoder_factory.h',
+      ],
+    },
+    {
       'target_name': 'rent_a_codec',
       'type': 'static_library',
       'defines': [
         '<@(audio_codec_defines)',
       ],
       'dependencies': [
         '<(webrtc_root)/common.gyp:webrtc_common',
         '<@(audio_codec_dependencies)',
       ],
       'include_dirs': [
@@ skipping 135 matching lines @@
           'sources': [
              'test/insert_packet_with_timing.cc',
              'test/Channel.cc',
              'test/PCMFile.cc',
            ],
         }, # delay_test
       ],
     }],
   ],
 }