summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKai Shi <kaishi@google.com>2020-05-27 23:50:28 +0000
committerAndroid (Google) Code Review <android-gerrit@google.com>2020-05-27 23:50:28 +0000
commite1718d217e401c2da9029d01ec99e1b335de0bb4 (patch)
tree77e76f17c79f34e0eb6202b32f81e497ad5e012c
parentc11cd31f25516662ea9a8d6b774d9e591a3fda91 (diff)
parent95e8ce98d9cc386e75b5a310f0829be7657ba3c1 (diff)
Merge "Add isCellularDataAvailable in sta event" into rvc-dev
-rw-r--r--service/java/com/android/server/wifi/WifiMetrics.java4
-rw-r--r--service/proto/src/metrics.proto3
-rw-r--r--tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java2
3 files changed, 9 insertions, 0 deletions
diff --git a/service/java/com/android/server/wifi/WifiMetrics.java b/service/java/com/android/server/wifi/WifiMetrics.java
index 383659e4d..621a2fbc8 100644
--- a/service/java/com/android/server/wifi/WifiMetrics.java
+++ b/service/java/com/android/server/wifi/WifiMetrics.java
@@ -4696,6 +4696,9 @@ public class WifiMetrics {
staEvent.totalTxBytes = mFacade.getTotalTxBytes();
staEvent.totalRxBytes = mFacade.getTotalRxBytes();
staEvent.screenOn = mScreenOn;
+ if (mWifiDataStall != null) {
+ staEvent.isCellularDataAvailable = mWifiDataStall.isCellularDataAvailable();
+ }
mSupplicantStateChangeBitmask = 0;
mLastPollRssi = -127;
mLastPollFreq = -1;
@@ -4911,6 +4914,7 @@ public class WifiMetrics {
if (event.totalTxBytes > 0) sb.append(" totalTxBytes=").append(event.totalTxBytes);
if (event.totalRxBytes > 0) sb.append(" totalRxBytes=").append(event.totalRxBytes);
sb.append(" screenOn=").append(event.screenOn);
+ sb.append(" cellularData=").append(event.isCellularDataAvailable);
if (event.supplicantStateChangesBitmask != 0) {
sb.append(", ").append(supplicantStateChangesBitmaskToString(
event.supplicantStateChangesBitmask));
diff --git a/service/proto/src/metrics.proto b/service/proto/src/metrics.proto
index ad52af1f4..0c3346bef 100644
--- a/service/proto/src/metrics.proto
+++ b/service/proto/src/metrics.proto
@@ -1438,6 +1438,9 @@ message StaEvent {
// Whether screen is on when the event happens
optional bool screen_on = 24;
+
+ // Whether cellular data network is available
+ optional bool is_cellular_data_available = 25;
}
// Wi-Fi Aware metrics
diff --git a/tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java b/tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java
index 04730bbe3..e4e58c1df 100644
--- a/tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java
+++ b/tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java
@@ -2285,6 +2285,7 @@ public class WifiMetricsTest extends WifiBaseTest {
}
mTestLooper.dispatchAll();
wifiMetrics.setScreenState(true);
+ when(mWifiDataStall.isCellularDataAvailable()).thenReturn(true);
for (int i = 0; i < mTestStaLogInts.length; i++) {
int[] lia = mTestStaLogInts[i];
wifiMetrics.logStaEvent(lia[0], lia[1], lia[2] == 1 ? mTestWifiConfig : null);
@@ -2324,6 +2325,7 @@ public class WifiMetricsTest extends WifiBaseTest {
assertConfigInfoEqualsWifiConfig(
evs[7] == 1 ? mTestWifiConfig : null, event.configInfo);
assertEquals(true, event.screenOn);
+ assertEquals(true, event.isCellularDataAvailable);
j++;
}
assertEquals(mExpectedValues.length, j);