Add new ANA stats to GetStats() to count the number of actions taken by each controller.

webrtc/common_types.h

 /*
  *  Copyright (c) 2012 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.
  */
 
@@ skipping 415 matching lines @@
   int calls_to_silence_generator;  // Number of calls where silence generated,
                                    // and NetEq was disengaged from decoding.
   int calls_to_neteq;              // Number of calls to NetEq.
   int decoded_normal;  // Number of calls where audio RTP packet decoded.
   int decoded_plc;     // Number of calls resulted in PLC.
   int decoded_cng;  // Number of calls where comfort noise generated due to DTX.
   int decoded_plc_cng;  // Number of calls resulted where PLC faded to CNG.
   int decoded_muted_output;  // Number of calls returning a muted state output.
 };
 
+// Statistics related to Audio Network Adaptation.
+struct ANAStats {
+  ANAStats();
+  ANAStats(const ANAStats&);
+  ~ANAStats();

[hbos, 2017/09/04 08:49:54]

Is this a style-guide thing or can these be omitted, since they're all =
default?

[ossu, 2017/09/04 11:23:02]

Due to the Optional<> members, clang(-tidy) decides that the struct is complex
enough that the constructors / destructors mustn't be inline.

+  rtc::Optional<int> bitrate_action_counter;
+  rtc::Optional<int> channel_action_counter;
+  rtc::Optional<int> dtx_action_counter;
+  rtc::Optional<int> fec_action_counter;
+  rtc::Optional<int> frame_length_action_counter;

[hbos, 2017/09/04 08:49:54]

Please add comments explaining these.

Also, prefer one of the types int32_t, uint32_t, int64_t, uint64_t for integers
when working with stats. int or int32_t shouldn't matter in this case but I
prefer all new stats code to be explicit about the type like that because the
new api and spec is and stats are exported to other formats (javascript, jni,
etc).

[ivoc, 2017/09/04 15:48:00]

Added comments and changed these to uint32_t.

+};
+
 // ==================================================================
 // Video specific types
 // ==================================================================
 
 // TODO(nisse): Delete, and switch to fourcc values everywhere?
 // Supported video types.
 enum class VideoType {
   kUnknown,
   kI420,
   kIYUV,
@@ skipping 519 matching lines @@
   bool operator==(const RtpKeepAliveConfig& o) const {
     return timeout_interval_ms == o.timeout_interval_ms &&
            payload_type == o.payload_type;
   }
   bool operator!=(const RtpKeepAliveConfig& o) const { return !(*this == o); }
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_COMMON_TYPES_H_