commit | e71aa499d637c48b159dc4ee7638f0003789e409 | [log] [tgz] |
---|---|---|
author | Wade Guthrie <wdg@chromium.org> | Mon Sep 23 13:44:19 2013 -0700 |
committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | Tue Sep 24 01:32:40 2013 +0000 |
tree | 40fccb4958f237179ba3e50afe0be1023aa7409d | |
parent | 39a7beb18a0c24c9b73c3cc49008ccdca19f9ac2 [diff] |
shill: Removes unused code in metrics.cc, adds comment to wifi. While toiling in metrics.cc (NotifyDeviceConnectFinished, to be specific), I spied a couple lines that were unused. In a fit of efficiency, I removed them. At a nearby time, I noticed that, while background scan times were not recorded, it's ok that we leave them unrecorded. I thought a comment explaining why might reduce future consternation. I added such a comment. Lint also pointed out a missing space in existing code (in metrics.cc). Upon reflection, I attended to this oversight. BUG=None TEST=unittest Change-Id: I996909a1d8d8e126db3c6197c517b7b6521f9d6a Reviewed-on: https://chromium-review.googlesource.com/170309 Reviewed-by: Paul Stewart <pstew@chromium.org> Reviewed-by: Wade Guthrie <wdg@chromium.org> Commit-Queue: Wade Guthrie <wdg@chromium.org> Tested-by: Wade Guthrie <wdg@chromium.org>