From a0a1f8a8619c32968fd36453692f9046f27726da Mon Sep 17 00:00:00 2001 From: Bookatz Date: Mon, 17 Dec 2018 12:28:32 -0800 Subject: [PATCH] Statsd atoms: Add comment about Wifi Add comment explaining caveats about the Wifi atoms. Bug: 115639456 Test: It is just a comment. Change-Id: I0882ccbe21c59439c4a6a61fb665bc0bf4695006 --- cmds/statsd/src/atoms.proto | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/cmds/statsd/src/atoms.proto b/cmds/statsd/src/atoms.proto index 5df47cd045b03..5899e0cfbca12 100644 --- a/cmds/statsd/src/atoms.proto +++ b/cmds/statsd/src/atoms.proto @@ -874,6 +874,7 @@ message KernelWakeupReported { /** * Logs when Wifi is toggled on/off. + * Note that Wifi may still perform certain functions (e.g. location scanning) even when disabled. * * Logged from: * frameworks/base/services/core/java/com/android/server/am/BatteryStatsService.java @@ -888,7 +889,8 @@ message WifiEnabledStateChanged { /** * Logs when an app causes Wifi to run. In this context, 'to run' means to use Wifi Client Mode. - * TODO: Include support for Hotspot. + * TODO: Include support for Hotspot, perhaps by using an extra field to denote 'mode'. + * Note that Wifi Scanning is monitored separately in WifiScanStateChanged. * * Logged from: * frameworks/base/services/core/java/com/android/server/am/BatteryStatsService.java