commit 125efc8c59728b57964a82af55ee294e0ed450bf
author Rafael Higuera Silva <rafahs@google.com> Fri Dec 16 02:01:51 2022 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Fri Dec 16 02:01:51 2022 +0000
tree b626b71aefc51f5e7839faf910958768413c58b6
parent e350186be6f47c7bff56624cef11e85fe36ddcd3
parent 4ec2763f1ca2c89ffe12c527031cee21b1c351fc

Remove daily pull policy. am: 4ec2763f1c

Original change: https://googleplex-android-review.googlesource.com/c/platform/packages/services/Mms/+/20724733

Change-Id: I38d842dd58ac4c3093887948b2983f0470ed9acc
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

src/com/android/mms/service/metrics/MmsMetricsCollector.java

diff --git a/src/com/android/mms/service/metrics/MmsMetricsCollector.java b/src/com/android/mms/service/metrics/MmsMetricsCollector.java
index 1bec4ae..f20d61a 100644
--- a/src/com/android/mms/service/metrics/MmsMetricsCollector.java
+++ b/src/com/android/mms/service/metrics/MmsMetricsCollector.java
@@ -24,7 +24,6 @@
 import android.util.Log;
 import android.util.StatsEvent;
 
-import androidx.annotation.Nullable;
 import androidx.annotation.VisibleForTesting;
 
 import com.android.mms.IncomingMms;
@@ -56,10 +55,6 @@
      */
     private static final long MIN_COOLDOWN_MILLIS =
             DBG ? 10L * MILLIS_PER_SECOND : 23L * MILLIS_PER_HOUR;
-    private static final StatsManager.PullAtomMetadata POLICY_PULL_DAILY =
-            new StatsManager.PullAtomMetadata.Builder()
-                    .setCoolDownMillis(MIN_COOLDOWN_MILLIS)
-                    .build();
     private final PersistMmsAtomsStorage mStorage;
     private final StatsManager mStatsManager;
 
@@ -73,8 +68,8 @@
         mStorage = storage;
         mStatsManager = context.getSystemService(StatsManager.class);
         if (mStatsManager != null) {
-            registerAtom(INCOMING_MMS, POLICY_PULL_DAILY);
-            registerAtom(OUTGOING_MMS, POLICY_PULL_DAILY);
+            registerAtom(INCOMING_MMS);
+            registerAtom(OUTGOING_MMS);
             Log.d(TAG, "[MmsMetricsCollector]: registered atoms");
         } else {
             Log.e(TAG, "[MmsMetricsCollector]: could not get StatsManager, "
@@ -152,9 +147,10 @@
         }
     }
 
-    /** Registers a pulled atom ID {@code atomId} with optional {@code policy} for pulling. */
-    private void registerAtom(int atomId, @Nullable StatsManager.PullAtomMetadata policy) {
-        mStatsManager.setPullAtomCallback(atomId, policy, ConcurrentUtils.DIRECT_EXECUTOR, this);
+    /** Registers a pulled atom ID {@code atomId}. */
+    private void registerAtom(int atomId) {
+        mStatsManager.setPullAtomCallback(atomId, /* metadata= */ null,
+                ConcurrentUtils.DIRECT_EXECUTOR, this);
     }
 
     /** Returns the {@link PersistMmsAtomsStorage} backing the puller. */