commit 58a37f8bb8032c4bb2e7cc6bd00d28c1841e7acb
author Robert Shih <robertshih@google.com> Mon Apr 19 10:18:50 2021 -0700
committer Robert Shih <robertshih@google.com> Mon Apr 19 11:56:28 2021 -0700
tree bc1aec39b485a926ea12f225064fabc91e56f079
parent 9a37b24337bd24c008f3650ccbf2e2d832d16f2a

statsd_drm: skip empty pulled atoms

Bug: 184263266
Test: Local Testing of New Atoms/Metrics
Test: http://doc/1pY7g5Xl0_aZ8JC6chQWUg9LHeBsjf2rNtqdc62IMIo8
Change-Id: I789cf3fba8d1ce3ee7477026ee3bd5d57d70b601

services/mediametrics/statsd_drm.cpp

diff --git a/services/mediametrics/statsd_drm.cpp b/services/mediametrics/statsd_drm.cpp
index 27fd089..73b8872 100644
--- a/services/mediametrics/statsd_drm.cpp
+++ b/services/mediametrics/statsd_drm.cpp
@@ -188,6 +188,11 @@
     (void) item->getString("plugin_metrics", &plugin_metrics);
     const auto plugin_raw(base64DecodeNoPad(plugin_metrics));
 
+    if (serialized_metrics.size() == 0 && plugin_metrics.size() == 0) {
+        ALOGD("statsd_mediadrm_puller skipping empty entry");
+        return false;
+    }
+
     std::string vendor;
     (void) item->getString("vendor", &vendor);
     std::string description;