Move AudioDecoder and related stuff to the api/ directory

webrtc/modules/audio_coding/codecs/internal_builtin_audio_decoder_factory.cc

 /*
  *  Copyright (c) 2016 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.
  */
 
-#include "webrtc/modules/audio_coding/codecs/builtin_audio_decoder_factory.h"
+#include "webrtc/modules/audio_coding/codecs/internal_builtin_audio_decoder_factory.h"
 
+#include <memory>
 #include <vector>
 
 #include "webrtc/base/checks.h"
 #include "webrtc/base/optional.h"
 #include "webrtc/common_types.h"
 #include "webrtc/modules/audio_coding/codecs/cng/webrtc_cng.h"
 #include "webrtc/modules/audio_coding/codecs/g711/audio_decoder_pcm.h"
 #ifdef WEBRTC_CODEC_G722
 #include "webrtc/modules/audio_coding/codecs/g722/audio_decoder_g722.h"
 #endif
@@ skipping 146 matching lines @@
        }
      }},
 #endif
 };
 
 class BuiltinAudioDecoderFactory : public AudioDecoderFactory {
  public:
   std::vector<AudioCodecSpec> GetSupportedDecoders() override {
     static std::vector<AudioCodecSpec> specs = {
 #ifdef WEBRTC_CODEC_OPUS
-      { { "opus", 48000, 2, {
-            {"minptime", "10" },
-            {"useinbandfec", "1" }
-          }
-        }, false
-      },
+      {{"opus",
+        48000,
+        2,
+        { {"minptime", "10"},
+          { "useinbandfec",

[ossu, 2017/02/01 15:05:26]

This was highlighted in my CL that clang-formatted this line:
It can be just one line and without starting and ending whitespace.

[kwiberg-webrtc, 2017/02/01 20:15:03]

I don't disagree. But again, this line only got changed in this CL in the first
place because clang-format munches every line of moved files, and I let it
because I don't want to fight.

If you feel strongly about this particular case, I can fix it up manually and
look up the magic incantation that will cause clang-format to leave it alone.

[ossu, 2017/02/06 12:54:16]

I do feel quite strongly about this case. clang-format cannot decide if what's
represented here is a list or key-value pair. Since it's the latter, I think
keeping them both on the same line is more important than it would otherwise be.
Make sure the extra spaces get removed as well, I fixed this line up but only
partially and had to go back afterwards[1].

[1]
https://codereview.webrtc.org/2669583002/diff2/1:20001/webrtc/modules/audio_c...

[kwiberg-webrtc, 2017/02/06 14:19:36]

OK, I'll make sure this is lovingly hand-formatted...

[ossu, 2017/02/06 14:44:06]

Artisanal lines of code - I like it!

+            "1" } }},
+       false},
 #endif
 #if (defined(WEBRTC_CODEC_ISAC) || defined(WEBRTC_CODEC_ISACFX))
       {{"isac", 16000, 1}, true},
 #endif
 #if (defined(WEBRTC_CODEC_ISAC))
       {{"isac", 32000, 1}, true},
 #endif
 #ifdef WEBRTC_CODEC_G722
       {{"G722", 8000, 1}, true},
 #endif
@@ skipping 32 matching lines @@
         }
         return decoder;
       }
     }
     return nullptr;
   }
 };
 
 }  // namespace
 
-rtc::scoped_refptr<AudioDecoderFactory> CreateBuiltinAudioDecoderFactory() {
+rtc::scoped_refptr<AudioDecoderFactory>
+InternalCreateBuiltinAudioDecoderFactory() {
   return rtc::scoped_refptr<AudioDecoderFactory>(
       new rtc::RefCountedObject<BuiltinAudioDecoderFactory>);
 }
 
 }  // namespace webrtc