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

remoting/client/jni/chromoting_jni_runtime.h

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef REMOTING_CLIENT_JNI_CHROMOTING_JNI_RUNTIME_H_
 #define REMOTING_CLIENT_JNI_CHROMOTING_JNI_RUNTIME_H_
 
 #include <jni.h>
 #include <string>
 
@@ skipping 61 matching lines @@
                               protocol::ErrorCode error);
 
   // Pops up a dialog box asking the user to enter a PIN. Call on UI thread.
   void DisplayAuthenticationPrompt(bool pairing_supported);
 
   // Saves new pairing credentials to permanent storage. Call on UI thread.
   void CommitPairingCredentials(const std::string& host,
                                 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);
+  // Updates the video image buffer, copying pixels from |buffer| into a Java
+  // Bitmap object. |buffer| is owned by the caller, and may be freed after this
+  // function returns. Call on the display thread.
+  void UpdateImageBuffer(int width, int height, uint8* buffer);

[Sergey Ulanov, 2013/09/25 23:32:08]

Pass the frame as webrtc::DesktopFrame reference. Then you wouldn't need the
comment about ownership.

 
   // Updates cursor shape. Call on display thread.
   void UpdateCursorShape(const protocol::CursorShapeInfo& cursor_shape);
 
   // Draws the latest image buffer onto the canvas. Call on the display thread.
   void RedrawCanvas();
 
  private:
   ChromotingJniRuntime();
 
@@ skipping 28 matching lines @@
   scoped_refptr<ChromotingJniInstance> session_;
 
   friend struct DefaultSingletonTraits<ChromotingJniRuntime>;
 
   DISALLOW_COPY_AND_ASSIGN(ChromotingJniRuntime);
 };
 
 }  // namespace remoting
 
 #endif