Format /api folder

At Meta, we have formatted the repo according to WebRTC .clang-format
file. Currently, those changes are stored as patch and we'd like to
apply them to the base WebRTC release instead.

I will be submitting CLs per folder. The plan is to format all h|cc|mm|m
files, while exlcuding Matlab files from the formatter as clang
misinterprets them as ObjC.

Formatting done via:

git ls-files | grep -E '^api\/.*\.(h|cc)' | xargs clang-format -i

No-Iwyu: Includes didn't change and it isn't related to formatting
Bug: webrtc:42225392
Change-Id: I4d7470104983d5d32612f9347301354265fb34c8
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/373520
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#43671}

api/audio/audio_device.h

@@ -61,13 +61,11 @@ class AudioDeviceModule : public webrtc::RefCountInterface {
  public:
   // Creates a default ADM for usage in production code.
   static rtc::scoped_refptr<AudioDeviceModule> Create(
-      AudioLayer audio_layer,
+      AudioLayer audio_layer, TaskQueueFactory* task_queue_factory);
-      TaskQueueFactory* task_queue_factory);
   // Creates an ADM with support for extra test methods. Don't use this factory
   // in production code.
   static rtc::scoped_refptr<AudioDeviceModuleForTest> CreateForTest(
-      AudioLayer audio_layer,
+      AudioLayer audio_layer, TaskQueueFactory* task_queue_factory);
-      TaskQueueFactory* task_queue_factory);
 
   // Retrieve the currently utilized audio layer
   virtual int32_t ActiveAudioLayer(AudioLayer* audioLayer) const = 0;

api/audio_codecs/builtin_audio_decoder_factory.cc

@@ -59,8 +59,7 @@ rtc::scoped_refptr<AudioDecoderFactory> CreateBuiltinAudioDecoderFactory() {
       AudioDecoderOpus, NotAdvertised<AudioDecoderMultiChannelOpus>,
 #endif
 
-      AudioDecoderG722,
+      AudioDecoderG722, AudioDecoderG711, NotAdvertised<AudioDecoderL16>>();
-      AudioDecoderG711, NotAdvertised<AudioDecoderL16>>();
 }
 
 }  // namespace webrtc

api/audio_codecs/builtin_audio_encoder_factory.cc

@@ -66,8 +66,7 @@ rtc::scoped_refptr<AudioEncoderFactory> CreateBuiltinAudioEncoderFactory() {
       AudioEncoderOpus, NotAdvertised<AudioEncoderMultiChannelOpus>,
 #endif
 
-      AudioEncoderG722,
+      AudioEncoderG722, AudioEncoderG711, NotAdvertised<AudioEncoderL16>>();
-      AudioEncoderG711, NotAdvertised<AudioEncoderL16>>();
 }
 
 }  // namespace webrtc

api/audio_codecs/opus/audio_encoder_multi_channel_opus_config.h

@@ -12,6 +12,7 @@
 #define API_AUDIO_CODECS_OPUS_AUDIO_ENCODER_MULTI_CHANNEL_OPUS_CONFIG_H_
 
 #include <stddef.h>
+
 #include <vector>
 
 #include "rtc_base/system/rtc_export.h"

api/fec_controller.h

@@ -37,6 +37,7 @@ class VCMProtectionCallback {
   // 'retransmission_mode' is either a value of enum RetransmissionMode, or
   // computed with bitwise operators on values of enum RetransmissionMode.
   virtual void SetRetransmissionMode(int retransmission_mode) = 0;
+
  protected:
   virtual ~VCMProtectionCallback() {}
 };

api/rtc_event_log_output.h

@@ -11,7 +11,6 @@
 #ifndef API_RTC_EVENT_LOG_OUTPUT_H_
 #define API_RTC_EVENT_LOG_OUTPUT_H_
 
-
 #include "absl/strings/string_view.h"
 
 namespace webrtc {

api/test/create_network_emulation_manager.h

@@ -22,7 +22,7 @@ namespace webrtc {
 std::unique_ptr<NetworkEmulationManager> CreateNetworkEmulationManager(
     NetworkEmulationManagerConfig config = NetworkEmulationManagerConfig());
 
-[[deprecated("Use version with NetworkEmulationManagerConfig)")]]
+[[deprecated("Use version with NetworkEmulationManagerConfig)")]]  //
 std::unique_ptr<NetworkEmulationManager>
 CreateNetworkEmulationManager(
     TimeMode time_mode,

api/test/mock_rtpsender.h

@@ -45,7 +45,9 @@ class MockRtpSender : public RtpSenderInterface {
               track,
               (),
               (const, override));
-  MOCK_METHOD(rtc::scoped_refptr<DtlsTransportInterface>, dtls_transport, (),
+  MOCK_METHOD(rtc::scoped_refptr<DtlsTransportInterface>,
+              dtls_transport,
+              (),
               (const, override));
   MOCK_METHOD(uint32_t, ssrc, (), (const, override));
   MOCK_METHOD(cricket::MediaType, media_type, (), (const, override));

api/test/mock_transformable_audio_frame.h

@@ -33,7 +33,9 @@ class MockTransformableAudioFrame : public TransformableAudioFrameInterface {
   MOCK_METHOD(uint32_t, GetSsrc, (), (const, override));
   MOCK_METHOD(uint32_t, GetTimestamp, (), (const, override));
   MOCK_METHOD(std::string, GetMimeType, (), (const, override));
-  MOCK_METHOD(rtc::ArrayView<const uint32_t>, GetContributingSources, (),
+  MOCK_METHOD(rtc::ArrayView<const uint32_t>,
+              GetContributingSources,
+              (),
               (const, override));
   MOCK_METHOD(const std::optional<uint16_t>,
               SequenceNumber,

api/test/mock_video_decoder.h

@@ -64,8 +64,7 @@ class MockVideoDecoder : public VideoDecoder {
   MOCK_METHOD(bool, Configure, (const Settings& settings), (override));
   MOCK_METHOD(int32_t,
               Decode,
-              (const EncodedImage& input_image,
+              (const EncodedImage& input_image, int64_t render_time_ms),
-               int64_t render_time_ms),
               (override));
   MOCK_METHOD(int32_t,
               Decode,

api/video/video_stream_encoder_settings.h

@@ -11,7 +11,6 @@
 #ifndef API_VIDEO_VIDEO_STREAM_ENCODER_SETTINGS_H_
 #define API_VIDEO_VIDEO_STREAM_ENCODER_SETTINGS_H_
 
-
 #include "api/video/video_bitrate_allocator_factory.h"
 #include "api/video_codecs/sdp_video_format.h"
 #include "api/video_codecs/video_encoder.h"

api/video_codecs/video_decoder.cc

@@ -47,9 +47,8 @@ std::string VideoDecoder::DecoderInfo::ToString() const {
   rtc::SimpleStringBuilder oss(string_buf);
 
   oss << "DecoderInfo { "
-      << "prefers_late_decoding = "
+      << "prefers_late_decoding = " << "implementation_name = '"
-      << "implementation_name = '" << implementation_name << "', "
+      << implementation_name << "', " << "is_hardware_accelerated = "
-      << "is_hardware_accelerated = "
       << (is_hardware_accelerated ? "true" : "false") << " }";
   return oss.str();
 }

api/voip/test/compile_all_headers.cc

@@ -10,4 +10,3 @@
 
 // This file verifies that all include files in this directory can be
 // compiled without errors or other required includes.
-