tree 40fccb4958f237179ba3e50afe0be1023aa7409d
parent 39a7beb18a0c24c9b73c3cc49008ccdca19f9ac2
author Wade Guthrie <wdg@chromium.org> 1379969059 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> 1379986360 +0000

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>