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

webrtc/modules/audio_coding/include/audio_coding_module.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 805 matching lines @@
   // to the time this API is called) will not be included in the list.
   //
   // Negative |round_trip_time_ms| results is an error message and empty list
   // is returned.
   //
   virtual std::vector<uint16_t> GetNackList(
       int64_t round_trip_time_ms) const = 0;
 
   virtual void GetDecodingCallStatistics(
       AudioDecodingCallStats* call_stats) const = 0;
+
+  virtual AudioEncoder::AudioEncoderStats GetAudioEncoderStatistics() const = 0;

[minyue-webrtc, 2017/08/31 07:24:21]

I think it is better to do similar to GetDecodingCallStatistics()

[ossu, 2017/08/31 13:13:10]

Why do you think so? Functions with output parameters are much more unwieldy to
call and only has a significant benefit in a few special cases (i.e. having it
fill in a (large) caller-managed buffer).

For this function, the copy could (and very probably will) be elided thanks to
Named Return Value Optimization.

[minyue-webrtc, 2017/08/31 13:20:17]

Copying is fine with me, and I understand the merits of not passing pointers.
What I would prefer, though is to let stats defined from top, since stats are
often defined in specs.

[ivoc, 2017/08/31 14:57:50]

I agree with ossu on return values vs passing pointers, but I'm not sure I
understand what minyue is arguing for. Minyue: do you think this function should
pass in the stats struct by pointer or that the stats struct should be defined
in common_types.h like AudioDecodingCallStats, or both?

[ivoc, 2017/09/01 15:27:16]

I moved the stats struct to common_types.h.

 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_AUDIO_CODING_INCLUDE_AUDIO_CODING_MODULE_H_