Move RtcEventLog object from inside VoiceEngine to 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

[the sun, 2016/03/03 09:25:13]

Remember that here you're changing an interface that is potentially used by 3rd
parties. How long have we had the GetEventLog() method? Is it reasonable to
assume nobody relies on it?

[ivoc, 2016/03/10 13:15:36]

It was introduced in Sept 2015, and is not currently in use in Google3 (as far
as I'm aware). It's not a terribly useful function for external users. I suppose
it could be used to start/stop the logging, but there are more convenient
functions on peerconnectionfactory to do that (which will move to peerconnection
in this CL).

-  // Engine. This pointer will remain valid until VoiceEngine is destroyed.
-  virtual RtcEventLog* GetEventLog() = 0;
-
  protected:
   VoECodec() {}
   virtual ~VoECodec() {}
 };
 
 }  // namespace webrtc
 
 #endif  //  WEBRTC_VOICE_ENGINE_VOE_CODEC_H