commit c53f952fb4df2d50d27b359280d3d8c8b02b64be
author Vova Sharaienko <sharaienko@google.com> Tue Feb 27 17:30:22 2024 +0000
committer Vova Sharaienko <sharaienko@google.com> Tue Feb 27 17:33:20 2024 +0000
tree 08165256fd9f705eaf7a21b192f8c568e6520450
parent 12e0eee1951453ebddb48098b8641221d35e85e8

Updated AggregatedMobileDataStatsPuller

- consider empty results as a valid case

Bug: 309512867
Test: adb shell cmd stats print-stats | grep 10204
Change-Id: Ic0f9cbea19d069414a55c606b164752105f33940

services/core/java/com/android/server/stats/pull/AggregatedMobileDataStatsPuller.java

diff --git a/services/core/java/com/android/server/stats/pull/AggregatedMobileDataStatsPuller.java b/services/core/java/com/android/server/stats/pull/AggregatedMobileDataStatsPuller.java
index b0dcf95..881583a 100644
--- a/services/core/java/com/android/server/stats/pull/AggregatedMobileDataStatsPuller.java
+++ b/services/core/java/com/android/server/stats/pull/AggregatedMobileDataStatsPuller.java
@@ -282,10 +282,7 @@
             Slog.d(TAG,
                     "pullDataBytesTransferLocked() done. results count " + pulledData.size());
         }
-        if (!pulledData.isEmpty()) {
-            return StatsManager.PULL_SUCCESS;
-        }
-        return StatsManager.PULL_SKIP;
+        return StatsManager.PULL_SUCCESS;
     }
 
     private static boolean isEmpty(NetworkStats stats) {