Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(9)

Unified Diff: webrtc/sdk/android/src/jni/native_handle_impl.h

Issue 2991633002: Implement AndroidVideoBuffer::ToI420. (Closed)
Patch Set: Rebase on top of remove-matrix. Created 3 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/sdk/android/src/jni/native_handle_impl.h
diff --git a/webrtc/sdk/android/src/jni/native_handle_impl.h b/webrtc/sdk/android/src/jni/native_handle_impl.h
index 560a84255cf2540b79c2ca24e74911a7d4aff4a8..9c4eb859bfc64b1e07a4f3ce7bf5465f1b753bc3 100644
--- a/webrtc/sdk/android/src/jni/native_handle_impl.h
+++ b/webrtc/sdk/android/src/jni/native_handle_impl.h
@@ -104,7 +104,6 @@ class AndroidTextureBuffer : public AndroidVideoFrameBuffer {
class AndroidVideoBuffer : public AndroidVideoFrameBuffer {
public:
AndroidVideoBuffer(JNIEnv* jni,
- jmethodID j_retain_id,
jmethodID j_release_id,
jmethodID j_get_width_id,
jmethodID j_get_height_id,
@@ -116,11 +115,15 @@ class AndroidVideoBuffer : public AndroidVideoFrameBuffer {
// Returns an instance of VideoRenderer.I420Frame (deprecated)
jobject ToJavaI420Frame(JNIEnv* jni, int rotation);
- private:
Type type() const override;
int width() const override;
int height() const override;
+ protected:
+ // Holds a VideoFrame.Buffer.
+ const ScopedGlobalRef<jobject> j_video_frame_buffer_;
+
+ private:
rtc::scoped_refptr<webrtc::I420BufferInterface> ToI420() override;
AndroidType android_type() override { return AndroidType::kJavaBuffer; }
@@ -128,8 +131,6 @@ class AndroidVideoBuffer : public AndroidVideoFrameBuffer {
const jmethodID j_release_id_;
const jmethodID j_get_width_id_;
const jmethodID j_get_height_id_;
- // Holds a VideoFrame.Buffer.
- ScopedGlobalRef<jobject> j_video_frame_buffer_;
};
class AndroidVideoBufferFactory {

Powered by Google App Engine
This is Rietveld 408576698