VideoCapturerAndroid, handle cvo correctly

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 96def5eae34e3b72cff636cc912bccca9b0eab4f..1c80882ef5cb02c2f27df053c122956722cb6001 100644
--- a/talk/app/webrtc/java/jni/androidvideocapturer_jni.h
+++ b/talk/app/webrtc/java/jni/androidvideocapturer_jni.h
@@ -60,7 +60,7 @@ class AndroidVideoCapturerJni : public webrtc::AndroidVideoCapturerDelegate {
   void OnCapturerStarted(bool success);
   void OnMemoryBufferFrame(void* video_frame, int length, int width,
                            int height, int rotation, int64_t timestamp_ns);
-  void OnTextureFrame(int width, int height, int64_t timestamp_ns,
+  void OnTextureFrame(int width, int height, int rotation, int64_t timestamp_ns,

[nisse-chromium (ooo August 14), 2015/12/08 08:46:34]

The rotation has to be one of the values in webrtc::VideoRotation, right?

It's a bit unclear in this interface if it is intended as an arbitrary angle
(and if so, what unit and direction), which happens to be only partially
implemented, or if it should be treated more like an enum.

I think it needs a comment, and some error handling in case an unexpected value
is received.

[perkj_chrome, 2015/12/09 21:00:40]

It is the same as OnMemoryBufferFrame and  is checked in
AndroidVideoCapturer::FrameFactory::UpdateCapturedFrame

[nisse-chromium (ooo August 14), 2015/12/10 09:04:57]

I see.

                       const NativeHandleImpl& handle);
   void OnOutputFormatRequest(int width, int height, int fps);