diff options
author | Ray Essick <essick@google.com> | 2019-01-28 20:34:42 -0800 |
---|---|---|
committer | Ray Essick <essick@google.com> | 2019-01-28 20:34:42 -0800 |
commit | e9b9c98a573d000abecc6d5a1718a78c06fff551 (patch) | |
tree | 2f2abe6e64b42ed45ab83cfc2f9a4374f5088733 /media/jni/android_media_MediaPlayer.cpp | |
parent | 5e55e809a722f1540630062d356c504169b3a688 (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.cpp | 8 |
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; } |