Move RtcEventLog object from inside VoiceEngine to Call.

webrtc/api/java/src/org/webrtc/PeerConnectionFactory.java

Index: webrtc/api/java/src/org/webrtc/PeerConnectionFactory.java
diff --git a/webrtc/api/java/src/org/webrtc/PeerConnectionFactory.java b/webrtc/api/java/src/org/webrtc/PeerConnectionFactory.java
index 17fc7ab8e86a90786ba919487cc1d0e4df1cbfa8..879abadbba68bb306913a5c9737555543ac88268 100644
--- a/webrtc/api/java/src/org/webrtc/PeerConnectionFactory.java
+++ b/webrtc/api/java/src/org/webrtc/PeerConnectionFactory.java
@@ -145,19 +145,6 @@ public class PeerConnectionFactory {
     nativeStopAecDump(nativeFactory);
   }
 
-  // Starts recording an RTC event log. Ownership of the file is transfered to
-  // the native code. If an RTC event log is already being recorded, it will be
-  // stopped and a new one will start using the provided file.
-  public boolean startRtcEventLog(int file_descriptor) {

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

Do you need to leave these as dummy methods to avoid the build breaking?

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

I think these are not used anywhere from the java interface. Chrome does use the
function but calls the C++ function directly.

-    return nativeStartRtcEventLog(nativeFactory, file_descriptor);
-  }
-
-  // Stops recording an RTC event log. If no RTC event log is currently being
-  // recorded, this call will have no effect.
-  public void StopRtcEventLog() {
-    nativeStopRtcEventLog(nativeFactory);
-  }
-
   @Deprecated
   public void setOptions(Options options) {
     nativeSetOptions(nativeFactory, options);
@@ -255,10 +242,6 @@ public class PeerConnectionFactory {
 
   private static native void nativeStopAecDump(long nativeFactory);
 
-  private static native boolean nativeStartRtcEventLog(long nativeFactory, int file_descriptor);
-
-  private static native void nativeStopRtcEventLog(long nativeFactory);
-
   @Deprecated
   public native void nativeSetOptions(long nativeFactory, Options options);