Add JNI interface for functions to start and stop recording AEC dumps and RTC event logs.

talk/app/webrtc/java/jni/peerconnection_jni.cc

 /*
  * libjingle
  * Copyright 2013 Google Inc.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are met:
  *
  *  1. Redistributions of source code must retain the above copyright notice,
  *     this list of conditions and the following disclaimer.
  *  2. Redistributions in binary form must reproduce the above copyright notice,
@@ skipping 1233 matching lines @@
     JNIEnv* jni, jclass, jlong native_factory, jstring id,
     jlong native_source) {
   rtc::scoped_refptr<PeerConnectionFactoryInterface> factory(
       factoryFromJava(native_factory));
   rtc::scoped_refptr<AudioTrackInterface> track(factory->CreateAudioTrack(
       JavaToStdString(jni, id),
       reinterpret_cast<AudioSourceInterface*>(native_source)));
   return (jlong)track.release();
 }
 
+JOW(jboolean, PeerConnectionFactory_nativeStartAecDump)(
+    JNIEnv* jni, jclass, jlong native_factory, jint file) {

[terelius, 2015/11/10 16:40:00]

Can the file handle safely be cast to a jint and back?

[jiayl1, 2015/11/10 17:00:04]

Seems OK with
http://developer.android.com/reference/android/os/ParcelFileDescriptor.html#d...

Will the native code close the handle?

[terelius, 2015/11/10 17:15:11]

Ok.

The RtcEventLog calls

FILE* file_stream = rtc::FdopenPlatformFileForWriting(platform_file_);
file_->OpenFromFileHandle(file_stream, true, false);

in StartLogging, and 

file_->CloseFile();
rtc::ClosePlatformFile(platform_file_);

in StopLogging. Ivo, do the AECdump do the same?

[ivoc, 2015/11/10 17:44:16]

@terelius: Good question, I think I need to add an #if defined(ANDROID) around
these function calls. On linux, the platform file is an int, but on windows it
is a HANDLE, which seems to be a pointer type (and therefore 64-bit), so this
conversion will probably cause compiler problems. As far as I'm aware, there is
no way to get access to a file descriptor (as int) in plain (non-Android) java
code anyway, so this restriction shouldn't matter much.

@terelius2: Yes, AEC dump does the same thing.

@jiayl: Yes, native code will close the handle.

[ivoc, 2015/11/10 17:44:16]

@terelius: Good question, I think I need to add an #if defined(ANDROID) around
these function calls. On linux, the platform file is an int, but on windows it
is a HANDLE, which seems to be a pointer type (and therefore 64-bit), so this
conversion will probably cause compiler problems. As far as I'm aware, there is
no way to get access to a file descriptor (as int) in plain (non-Android) java
code anyway, so this restriction shouldn't matter much.

@jiayl: Yes, native code will close the handle.

+  rtc::scoped_refptr<PeerConnectionFactoryInterface> factory(
+      factoryFromJava(native_factory));
+  return factory->StartAecDump(file);
+}
+
+JOW(void, PeerConnectionFactory_nativeStopAecDump)(
+    JNIEnv* jni, jclass, jlong native_factory) {
+  rtc::scoped_refptr<PeerConnectionFactoryInterface> factory(
+      factoryFromJava(native_factory));
+  factory->StopAecDump();
+}
+
+JOW(jboolean, PeerConnectionFactory_nativeStartRtcEventLog)(
+    JNIEnv* jni, jclass, jlong native_factory, jint file) {
+  rtc::scoped_refptr<PeerConnectionFactoryInterface> factory(
+      factoryFromJava(native_factory));
+  return factory->StartRtcEventLog(file);
+}
+
+JOW(void, PeerConnectionFactory_nativeStopRtcEventLog)(
+    JNIEnv* jni, jclass, jlong native_factory) {
+  rtc::scoped_refptr<PeerConnectionFactoryInterface> factory(
+      factoryFromJava(native_factory));
+  factory->StopRtcEventLog();
+}
+
 JOW(void, PeerConnectionFactory_nativeSetOptions)(
     JNIEnv* jni, jclass, jlong native_factory, jobject options) {
   rtc::scoped_refptr<PeerConnectionFactoryInterface> factory(
       factoryFromJava(native_factory));
   jclass options_class = jni->GetObjectClass(options);
   jfieldID network_ignore_mask_field =
       jni->GetFieldID(options_class, "networkIgnoreMask", "I");
   int network_ignore_mask =
       jni->GetIntField(options, network_ignore_mask_field);
 
@@ skipping 782 matching lines @@
   return JavaStringFromStdString(
       jni,
       reinterpret_cast<RtpReceiverInterface*>(j_rtp_receiver_pointer)->id());
 }
 
 JOW(void, RtpReceiver_free)(JNIEnv* jni, jclass, jlong j_rtp_receiver_pointer) {
   reinterpret_cast<RtpReceiverInterface*>(j_rtp_receiver_pointer)->Release();
 }
 
 }  // namespace webrtc_jni