Hold video frame in Bitmap instead of keeping a ByteBuffer reference.

remoting/client/jni/chromoting_jni_runtime.h

Index: remoting/client/jni/chromoting_jni_runtime.h
diff --git a/remoting/client/jni/chromoting_jni_runtime.h b/remoting/client/jni/chromoting_jni_runtime.h
index bfe85492a6b62e995f80d418d9523ebde6d1bb17..252d2179e015cb12f4c2440ee163907babfa8a1b 100644
--- a/remoting/client/jni/chromoting_jni_runtime.h
+++ b/remoting/client/jni/chromoting_jni_runtime.h
@@ -8,6 +8,7 @@
 #include <jni.h>
 #include <string>
 
+#include "base/android/scoped_java_ref.h"
 #include "base/at_exit.h"
 #include "net/url_request/url_request_context_getter.h"
 #include "remoting/base/auto_thread.h"
@@ -16,6 +17,10 @@
 
 template<typename T> struct DefaultSingletonTraits;
 
+namespace webrtc {
+class DesktopFrame;

[Sergey Ulanov, 2013/10/08 22:15:02]

Not used in this file.

[Lambros, 2013/10/10 01:35:58]

Removed.

+}  // namespace webrtc
+
 namespace remoting {
 
 // Houses the global resources on which the Chromoting components run
@@ -79,8 +84,12 @@ class ChromotingJniRuntime {
                                 const std::string& id,
                                 const std::string& secret);
 
-  // Updates image dimensions and canvas memory space. Call on display thread.
-  void UpdateImageBuffer(int width, int height, jobject buffer);
+  // Creates a new Bitmap object to store a video frame.
+  base::android::ScopedJavaLocalRef<jobject> NewBitmap(
+      webrtc::DesktopSize size);
+
+  // Updates video frame bitmap. Call on the display thread.
+  void UpdateFrameBitmap(jobject bitmap);

[Sergey Ulanov, 2013/10/08 22:15:02]

please comment that |bitmat| must be an android.graphics.Bitmap.

[Lambros, 2013/10/10 01:35:58]

Done.

 
   // Updates cursor shape. Call on display thread.
   void UpdateCursorShape(const protocol::CursorShapeInfo& cursor_shape);