commit | 220ae4e84acc9c7edead58c719b727b5fea925ff | [log] [tgz] |
---|---|---|
author | Paul Stewart <pstew@chromium.org> | Thu Feb 12 13:04:36 2015 -0800 |
committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | Fri Feb 13 06:48:50 2015 +0000 |
tree | 68f37621438470572252edf15c52193bf291aa30 | |
parent | 762bfb8ab200a387fe732ec92423a5f0afe11bcf [diff] |
shill: Manager: Limit log messages for service state changes At the normal debug log level, only report service state in Manager::UpdateService under 2 conditions: - An inactive service becomes active - An active service changes states (including to an inactive state). Notably, this removes log messages when other attributes of the service change (for example signal strength of WiFi services). To do this, track the states of all watched (active) services, and use this to decide whether to output at the normal INFO log level or at a debugging level. BUG=chromium:458224 TEST=New unit test. Change-Id: Id2fb7075854c34d8f1d08095d573c5e5e66d39fb Reviewed-on: https://chromium-review.googlesource.com/249420 Reviewed-by: Rebecca Silberstein <silberst@chromium.org> Commit-Queue: Paul Stewart <pstew@chromium.org> Tested-by: Paul Stewart <pstew@chromium.org>