summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Mathias Agopian <mathias@google.com> 2017-04-03 17:51:15 -0700
committer Mathias Agopian <mathias@google.com> 2017-04-03 17:51:15 -0700
commit845eef05ffea53f4ce7a1e0551896be874c4f302 (patch)
tree8e64bc698fa312fe3176c071dae91d316adac900
parent64731b1d41eccdf53450195d63ff9e3bd8c57fd7 (diff)
Remove useless GraphicBuffer() constructor
Test: compiled & run Bug: 36869708 Change-Id: Ie825979cd03874904b6f953ece4a05d4a0d52253
-rw-r--r--core/jni/android_hardware_camera2_legacy_LegacyCameraDevice.cpp2
-rw-r--r--media/jni/android_media_ImageWriter.cpp2
2 files changed, 2 insertions, 2 deletions
diff --git a/core/jni/android_hardware_camera2_legacy_LegacyCameraDevice.cpp b/core/jni/android_hardware_camera2_legacy_LegacyCameraDevice.cpp
index 80f9d5750d8a..71211948dac3 100644
--- a/core/jni/android_hardware_camera2_legacy_LegacyCameraDevice.cpp
+++ b/core/jni/android_hardware_camera2_legacy_LegacyCameraDevice.cpp
@@ -182,7 +182,7 @@ static status_t produceFrame(const sp<ANativeWindow>& anw,
err = native_window_dequeue_buffer_and_wait(anw.get(), &anb);
if (err != NO_ERROR) return err;
- sp<GraphicBuffer> buf(new GraphicBuffer(anb, /*keepOwnership*/false));
+ sp<GraphicBuffer> buf(GraphicBuffer::from(anb));
uint32_t grallocBufWidth = buf->getWidth();
uint32_t grallocBufHeight = buf->getHeight();
uint32_t grallocBufStride = buf->getStride();
diff --git a/media/jni/android_media_ImageWriter.cpp b/media/jni/android_media_ImageWriter.cpp
index d5d9fc94bf29..0149d7623660 100644
--- a/media/jni/android_media_ImageWriter.cpp
+++ b/media/jni/android_media_ImageWriter.cpp
@@ -342,7 +342,7 @@ static void ImageWriter_dequeueImage(JNIEnv* env, jobject thiz, jlong nativeCtx,
}
// New GraphicBuffer object doesn't own the handle, thus the native buffer
// won't be freed when this object is destroyed.
- sp<GraphicBuffer> buffer(new GraphicBuffer(anb, /*keepOwnership*/false));
+ sp<GraphicBuffer> buffer(GraphicBuffer::from(anb));
// Note that:
// 1. No need to lock buffer now, will only lock it when the first getPlanes() is called.