commit 5e45c31d0eb761beacd1a3f7bd55cc802a1f1d93
author Robert Shih <robertshih@google.com> Thu Apr 22 16:27:43 2021 -0700
committer Robert Shih <robertshih@google.com> Fri Apr 23 15:57:19 2021 -0700
tree 535b0d2ea5db0e089efee63936c980587af61c0e
parent 3665bc2c79a0ae166df57a92a301b61751f47aec

statsd_drm pushed atom: skip empty entries

Bug: 141714243
Test: statsd_testdrive
Change-Id: I521c7482dd0c157c9ad25f45d6bbfa391a9e562c

services/mediametrics/statsd_drm.cpp

diff --git a/services/mediametrics/statsd_drm.cpp b/services/mediametrics/statsd_drm.cpp
index 73b8872..287fb8d 100644
--- a/services/mediametrics/statsd_drm.cpp
+++ b/services/mediametrics/statsd_drm.cpp
@@ -59,9 +59,16 @@
     std::string description;
     (void) item->getString("description", &description);
 
+    std::string serialized_metrics;
+    (void) item->getString("serialized_metrics", &serialized_metrics);
+    if (serialized_metrics.empty()) {
+        ALOGD("statsd_mediadrm skipping empty entry");
+        return false;
+    }
+
     // This field is left here for backward compatibility.
     // This field is not used anymore.
-    const std::string  kUnusedField("unused");
+    const std::string  kUnusedField("");
     android::util::BytesField bf_serialized(kUnusedField.c_str(), kUnusedField.size());
     int result = android::util::stats_write(android::util::MEDIAMETRICS_MEDIADRM_REPORTED,
         timestamp_nanos, package_name.c_str(), package_version_code,