Hooked up RtcEventLog. It lives in Voice Engine and pointers are propagated to ACM and Call.

webrtc/voice_engine/include/voe_codec.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 17 matching lines @@
 //  codec->Release();
 //  VoiceEngine::Delete(voe);
 //
 #ifndef WEBRTC_VOICE_ENGINE_VOE_CODEC_H
 #define WEBRTC_VOICE_ENGINE_VOE_CODEC_H
 
 #include "webrtc/common_types.h"
 
 namespace webrtc {
 
+class RtcEventLog;
 class VoiceEngine;
 
 class WEBRTC_DLLEXPORT VoECodec {
  public:
   // Factory for the VoECodec sub-API. Increases an internal
   // reference counter if successful. Returns NULL if the API is not
   // supported or if construction fails.
   static VoECodec* GetInterface(VoiceEngine* voiceEngine);
 
   // Releases the VoECodec sub-API and decreases an internal
@@ skipping 76 matching lines @@
   // TODO(minyue): Make SetOpusMaxPlaybackRate() pure virtual when
   // fakewebrtcvoiceengine in talk is ready.
   virtual int SetOpusMaxPlaybackRate(int channel, int frequency_hz) {
     return -1;
   }
 
   // If send codec is Opus on a specified |channel|, set its DTX. Returns 0 if
   // success, and -1 if failed.
   virtual int SetOpusDtx(int channel, bool enable_dtx) = 0;
 
+  // Get a pointer to the event logging object associated with this Voice

[Henrik Grunell WebRTC, 2015/08/05 09:44:35]

Is the event logging object related to codec(s) only? Should this function
perhaps be in another "sub-API"?

[ivoc, 2015/08/06 10:57:51]

I was not entirely sure which of the sub-APIs is most appropriate for this
functionality. Initially I chose VoEBase, but Minyue@ said VoECodec would be
more appropriate because the actual logging is done in the ACM, and most of the
functions that relate to ACM are in VoECodec. If you have a better idea, feel
free to share.

[Henrik Grunell WebRTC, 2015/08/06 15:30:45]

Right, it's given to |audio_coding_| in the channel. This seems to be the right
place.

+  // Engine.
+  virtual RtcEventLog* GetEventLog() = 0;

[Henrik Grunell WebRTC, 2015/08/05 09:44:35]

Instead of getting a pointer and operate directly on that object, add functions
here that do what we need. E.g. Enable, Disable. The internal object should be
hidden.

[ivoc, 2015/08/06 10:57:52]

We need more functionality than just enable/disable. The pointer that is
obtained from this function is used in Call.cc to log RTP packets, RTCP packets
and video send/receive configs. We would have to basically replicate the whole
interface from the rtc_event_log.h header here, is that really desirable? 

[Henrik Grunell WebRTC, 2015/08/06 15:30:45]

OK. In that case I'd prefer the VoE user to have ownership of that object and
pass it to VoE. Let'd discuss offline tomorrow.

[Henrik Grunell WebRTC, 2015/08/07 11:30:03]

We talked offline and since (1) the path in libjingle between VoE and Call is
quite long with many layers, (2) the event log will eventually be moved
completely to Call, and (3) VoE outlives Call, I think it's OK to do it this
way.

+
  protected:
   VoECodec() {}
   virtual ~VoECodec() {}
 };
 
 }  // namespace webrtc
 
 #endif  //  WEBRTC_VOICE_ENGINE_VOE_CODEC_H