passing |buffer| by reference in AndroidVideoCapturer::OnIncomingFrame

talk/app/webrtc/java/jni/androidvideocapturer_jni.h

Index: talk/app/webrtc/java/jni/androidvideocapturer_jni.h
diff --git a/talk/app/webrtc/java/jni/androidvideocapturer_jni.h b/talk/app/webrtc/java/jni/androidvideocapturer_jni.h
index 360674bb70c00f79670357b81429328b39d9cf26..44abf6607a5bac727168f1e9015e9be8ac42a63e 100644
--- a/talk/app/webrtc/java/jni/androidvideocapturer_jni.h
+++ b/talk/app/webrtc/java/jni/androidvideocapturer_jni.h
@@ -71,13 +71,16 @@ class AndroidVideoCapturerJni : public webrtc::AndroidVideoCapturerDelegate {
   void ReturnBuffer(int64_t time_stamp);
   JNIEnv* jni();
 
+  // To avoid deducing Args from the 3rd parameter of AsyncCapturerInvoke
+  template <typename T> struct Identity  { typedef T type;  };

[magjed_webrtc, 2015/10/20 13:06:23]

It this necessary to make it compile now?
nit: Remove extra space after 'typedef T type'. You can also use 'git cl format'
to fix stuff like that automatically.

+
   // Helper function to make safe asynchronous calls to |capturer_|. The calls
   // are not guaranteed to be delivered.
   template <typename... Args>
   void AsyncCapturerInvoke(
       const char* method_name,
       void (webrtc::AndroidVideoCapturer::*method)(Args...),
-      Args... args);
+      typename Identity<Args>::type... args);
 
   const ScopedGlobalRef<jobject> j_capturer_global_;
   const ScopedGlobalRef<jclass> j_video_capturer_class_;