diff options
author | Kai Shi <kaishi@google.com> | 2020-05-27 23:50:28 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2020-05-27 23:50:28 +0000 |
commit | e1718d217e401c2da9029d01ec99e1b335de0bb4 (patch) | |
tree | 77e76f17c79f34e0eb6202b32f81e497ad5e012c | |
parent | c11cd31f25516662ea9a8d6b774d9e591a3fda91 (diff) | |
parent | 95e8ce98d9cc386e75b5a310f0829be7657ba3c1 (diff) |
Merge "Add isCellularDataAvailable in sta event" into rvc-dev
-rw-r--r-- | service/java/com/android/server/wifi/WifiMetrics.java | 4 | ||||
-rw-r--r-- | service/proto/src/metrics.proto | 3 | ||||
-rw-r--r-- | tests/wifitests/src/com/android/server/wifi/WifiMetricsTest.java | 2 |
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); |