summaryrefslogtreecommitdiff
path: root/core/java/android
diff options
context:
space:
mode:
authorRoshan Pius <rpius@google.com>2020-03-05 10:34:24 -0800
committerRoshan Pius <rpius@google.com>2020-03-05 10:34:24 -0800
commitcdf5744a77ce99f2095d7640e3715746dd5d7510 (patch)
tree41424fbdfc07c06aab695ae63d81e0b7660df29a /core/java/android
parentd1bbc14732b9d4b42d73ed3f176c2dac6f381ed7 (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.java14
-rw-r--r--core/java/android/os/connectivity/WifiBatteryStats.java4
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;