summaryrefslogtreecommitdiff
path: root/media/jni/android_media_MediaExtractor.cpp
diff options
context:
space:
mode:
authorRay Essick <essick@google.com>2019-01-28 20:34:42 -0800
committerRay Essick <essick@google.com>2019-01-28 20:34:42 -0800
commite9b9c98a573d000abecc6d5a1718a78c06fff551 (patch)
tree2f2abe6e64b42ed45ab83cfc2f9a4374f5088733 /media/jni/android_media_MediaExtractor.cpp
parent5e55e809a722f1540630062d356c504169b3a688 (diff)
use MediaAnalyticsItem::create() instead of new
Bug: 123543489 Test: boot/logcat, select CtsMediaTests
Diffstat (limited to 'media/jni/android_media_MediaExtractor.cpp')
-rw-r--r--media/jni/android_media_MediaExtractor.cpp8
1 files changed, 2 insertions, 6 deletions
diff --git a/media/jni/android_media_MediaExtractor.cpp b/media/jni/android_media_MediaExtractor.cpp
index f3442f4b4d33..c6b171bdd6d4 100644
--- a/media/jni/android_media_MediaExtractor.cpp
+++ b/media/jni/android_media_MediaExtractor.cpp
@@ -913,13 +913,9 @@ android_media_MediaExtractor_native_getMetrics(JNIEnv * env, jobject thiz)
}
// build and return the Bundle
- MediaAnalyticsItem *item = new MediaAnalyticsItem;
+ std::unique_ptr<MediaAnalyticsItem> item(MediaAnalyticsItem::create());
item->readFromParcel(reply);
- jobject mybundle = MediaMetricsJNI::writeMetricsToBundle(env, item, NULL);
-
- // housekeeping
- delete item;
- item = NULL;
+ jobject mybundle = MediaMetricsJNI::writeMetricsToBundle(env, item.get(), NULL);
return mybundle;
}