Fix DrmHal implementation to return all metrics

This change in DrmHal allows us to transfer a PersistableBundle containing
previously unavailable metrics.

Bug: 64001676
Test: CTS test for metrics and GPlay test
Change-Id: I1fa87c76dd980a3f91b4e2d02b37329f6c0a88ec

1 files changed, 5 insertions, 5 deletions

diff --git a/drm/libmediadrm/IDrm.cpp b/drm/libmediadrm/IDrm.cpp
index 63a95626a2..802e16fdad 100644
--- a/drm/libmediadrm/IDrm.cpp
+++ b/drm/libmediadrm/IDrm.cpp
@@ -508,7 +508,7 @@ struct BpDrm : public BpInterface<IDrm> {
         return reply.readInt32();
     }
 
-    virtual status_t getMetrics(MediaAnalyticsItem *item) {
+    virtual status_t getMetrics(os::PersistableBundle *metrics) {
         Parcel data, reply;
         data.writeInterfaceToken(IDrm::getInterfaceDescriptor());
 
@@ -517,7 +517,7 @@ struct BpDrm : public BpInterface<IDrm> {
             return status;
         }
 
-        item->readFromParcel(reply);
+        metrics->readFromParcel(&reply);
         return reply.readInt32();
     }
 
@@ -1034,9 +1034,9 @@ status_t BnDrm::onTransact(
         {
             CHECK_INTERFACE(IDrm, data, reply);
 
-            MediaAnalyticsItem item;
-            status_t result = getMetrics(&item);
-            item.writeToParcel(reply);
+            os::PersistableBundle metrics;
+            status_t result = getMetrics(&metrics);
+            metrics.writeToParcel(reply);
             reply->writeInt32(result);
             return OK;
         }