diff options
| author | Roshan Pius <rpius@google.com> | 2020-03-05 10:34:24 -0800 |
|---|---|---|
| committer | Roshan Pius <rpius@google.com> | 2020-03-05 10:34:24 -0800 |
| commit | cdf5744a77ce99f2095d7640e3715746dd5d7510 (patch) | |
| tree | 41424fbdfc07c06aab695ae63d81e0b7660df29a /core/java/android | |
| parent | d1bbc14732b9d4b42d73ed3f176c2dac6f381ed7 (diff) | |
BatteryStatsManager: Address API council feedback
Bug: 144925883
Test: Compiles
Change-Id: Ibdc61769965a97222fdd968084eca71b997e8feb
Diffstat (limited to 'core/java/android')
| -rw-r--r-- | core/java/android/os/BatteryStatsManager.java | 14 | ||||
| -rw-r--r-- | core/java/android/os/connectivity/WifiBatteryStats.java | 4 |
2 files changed, 10 insertions, 8 deletions
diff --git a/core/java/android/os/BatteryStatsManager.java b/core/java/android/os/BatteryStatsManager.java index f2e16b46422f..a9585c62866b 100644 --- a/core/java/android/os/BatteryStatsManager.java +++ b/core/java/android/os/BatteryStatsManager.java @@ -350,12 +350,13 @@ public final class BatteryStatsManager { /** * Indicates that an app has acquired the wifi multicast lock. * - * @param uid UID of the app that acquired the wifi lock (to be used for battery blaming). + * @param ws Worksource with the uid of the app that acquired the wifi lock (to be used for + * battery blaming). */ @RequiresPermission(android.Manifest.permission.UPDATE_DEVICE_STATS) - public void reportWifiMulticastEnabled(int uid) { + public void reportWifiMulticastEnabled(@NonNull WorkSource ws) { try { - mBatteryStats.noteWifiMulticastEnabled(uid); + mBatteryStats.noteWifiMulticastEnabled(ws.getAttributionUid()); } catch (RemoteException e) { e.rethrowFromSystemServer(); } @@ -364,12 +365,13 @@ public final class BatteryStatsManager { /** * Indicates that an app has released the wifi multicast lock. * - * @param uid UID of the app that released the wifi lock (to be used for battery blaming). + * @param ws Worksource with the uid of the app that released the wifi lock (to be used for + * battery blaming). */ @RequiresPermission(android.Manifest.permission.UPDATE_DEVICE_STATS) - public void reportWifiMulticastDisabled(int uid) { + public void reportWifiMulticastDisabled(@NonNull WorkSource ws) { try { - mBatteryStats.noteWifiMulticastDisabled(uid); + mBatteryStats.noteWifiMulticastDisabled(ws.getAttributionUid()); } catch (RemoteException e) { e.rethrowFromSystemServer(); } diff --git a/core/java/android/os/connectivity/WifiBatteryStats.java b/core/java/android/os/connectivity/WifiBatteryStats.java index 3c30f6343405..7e6ebcfc61db 100644 --- a/core/java/android/os/connectivity/WifiBatteryStats.java +++ b/core/java/android/os/connectivity/WifiBatteryStats.java @@ -215,7 +215,7 @@ public final class WifiBatteryStats implements Parcelable { * Returns the number of bytes transmitted over wifi within * {@link #getLoggingDurationMillis()}. * - * @return Number of packets received. + * @return Number of bytes transmitted. */ public long getNumBytesTx() { return mNumBytesTx; @@ -225,7 +225,7 @@ public final class WifiBatteryStats implements Parcelable { * Returns the number of packets received over wifi within * {@link #getLoggingDurationMillis()}. * - * @return Number of bytes transmitted. + * @return Number of packets received. */ public long getNumPacketsRx() { return mNumPacketsRx; |
