shill: Improves logging for scan.
Looking at the logs, it's hard to tell why shill chooses to do a scan.
This CL adds a 'reason' (which is really the name of the calling
function) to Scan calls to make it clear why we're scanning.
BUG=None
TEST=unittest
Change-Id: Ibd128625681fc5d61d359b60953bb78d89e97dfe
Reviewed-on: https://gerrit.chromium.org/gerrit/63357
Reviewed-by: mukesh agrawal <quiche@chromium.org>
Commit-Queue: Wade Guthrie <wdg@chromium.org>
Reviewed-by: Wade Guthrie <wdg@chromium.org>
Tested-by: Wade Guthrie <wdg@chromium.org>
diff --git a/cellular.cc b/cellular.cc
index 2896a32..b082a0b 100644
--- a/cellular.cc
+++ b/cellular.cc
@@ -438,7 +438,8 @@
}
}
-void Cellular::Scan(ScanType scan_type, Error *error) {
+void Cellular::Scan(ScanType /*scan_type*/, Error *error,
+ const string &/*reason*/) {
// |scan_type| is ignored because Cellular only does a full scan.
// TODO(ers): for now report immediate success or failure.
capability_->Scan(error, ResultCallback());