summaryrefslogtreecommitdiff
path: root/media/jni/android_media_MediaPlayer.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_MediaPlayer.cpp
parent5e55e809a722f1540630062d356c504169b3a688 (diff)
use MediaAnalyticsItem::create() instead of new
Bug: 123543489 Test: boot/logcat, select CtsMediaTests
Diffstat (limited to 'media/jni/android_media_MediaPlayer.cpp')
-rw-r--r--media/jni/android_media_MediaPlayer.cpp8
1 files changed, 2 insertions, 6 deletions
diff --git a/media/jni/android_media_MediaPlayer.cpp b/media/jni/android_media_MediaPlayer.cpp
index 76bbce7f0d87..35b10817c05c 100644
--- a/media/jni/android_media_MediaPlayer.cpp
+++ b/media/jni/android_media_MediaPlayer.cpp
@@ -682,13 +682,9 @@ android_media_MediaPlayer_native_getMetrics(JNIEnv *env, jobject thiz)
return (jobject) NULL;
}
- MediaAnalyticsItem *item = new MediaAnalyticsItem;
+ std::unique_ptr<MediaAnalyticsItem> item(MediaAnalyticsItem::create());
item->readFromParcel(p);
- jobject mybundle = MediaMetricsJNI::writeMetricsToBundle(env, item, NULL);
-
- // housekeeping
- delete item;
- item = NULL;
+ jobject mybundle = MediaMetricsJNI::writeMetricsToBundle(env, item.get(), NULL);
return mybundle;
}