commit 3c41610f310bd4e89880364e3b6eedba026470de
author Mingguang Xu <mingguangxu@google.com> Wed Feb 20 11:52:51 2019 -0800
committer Mingguang Xu <mingguangxu@google.com> Wed Feb 20 11:52:51 2019 -0800
tree 0c38003489925c33aa8686e219bd92eb4edb943c
parent e44179b88a1b40dce662fd17e0c141d547ca42a2

Check screen status before logging Wifi usability stats

Wifi usability stats triggered by FW/Driver alert event should be collected only when screen is on, otherwise, the stats collected will be obsolete.

Bug: 124916431

Test: frameworks/opt/net/wifi/tests/wifitests/runtests.sh

Change-Id: I99f74990fe405542a5e049c1b056188c68f5ce00
Signed-off-by: Mingguang Xu <mingguangxu@google.com>

service/java/com/android/server/wifi/WifiMetrics.java

diff --git a/service/java/com/android/server/wifi/WifiMetrics.java b/service/java/com/android/server/wifi/WifiMetrics.java
index 86b933c..fba7665 100644
--- a/service/java/com/android/server/wifi/WifiMetrics.java
+++ b/service/java/com/android/server/wifi/WifiMetrics.java
@@ -2058,8 +2058,10 @@
     public void logFirmwareAlert(int errorCode) {
         incrementAlertReasonCount(errorCode);
         logWifiIsUnusableEvent(WifiIsUnusableEvent.TYPE_FIRMWARE_ALERT, errorCode);
-        addToWifiUsabilityStatsList(WifiUsabilityStats.LABEL_BAD,
-                WifiUsabilityStats.TYPE_FIRMWARE_ALERT);
+        if (mScreenOn) {
+            addToWifiUsabilityStatsList(WifiUsabilityStats.LABEL_BAD,
+                    WifiUsabilityStats.TYPE_FIRMWARE_ALERT);
+        }
     }
 
     public static final String PROTO_DUMP_ARG = "wifiMetricsProto";