commit 4a66cc5e91ad82743ab196257438e3a0c313aab7
author Darin Petkov <petkov@chromium.org> Fri Jun 15 10:08:29 2012 +0200
committer Gerrit <chrome-bot@google.com> Fri Jun 15 14:34:56 2012 -0700
tree b1fe924a547da88430cc3283a92b4e61eab2ca0d
parent 3a4100c44c9676f17829de5bcb23e883bd970ef7

shill: wifi: Disable or reduce frequency of background scan.

For details, see corresponding flimflam patch:

https://gerrit.chromium.org/gerrit/#change,24960

Also, rename per style and fix a bug in NumEndpoints (returning bool
rather than an int) and add more unit tests.

BUG=chromium-os:31715
TEST=unit tests

Change-Id: I9b0c6f1abc660a44a43f34b1dec2230c6cf6cfde
Reviewed-on: https://gerrit.chromium.org/gerrit/25376
Reviewed-by: Darin Petkov <petkov@chromium.org>
Tested-by: Darin Petkov <petkov@chromium.org>
Commit-Ready: Darin Petkov <petkov@chromium.org>

wifi_service.cc

diff --git a/wifi_service.cc b/wifi_service.cc
index 0afa2e5..a6eafdd 100644
--- a/wifi_service.cc
+++ b/wifi_service.cc
@@ -175,13 +175,13 @@
   return wifi_->IsConnectingTo(*this);
 }
 
-void WiFiService::AddEndpoint(const WiFiEndpointConstRefPtr endpoint) {
+void WiFiService::AddEndpoint(const WiFiEndpointConstRefPtr &endpoint) {
   DCHECK(endpoint->ssid() == ssid());
   endpoints_.insert(endpoint);
   UpdateFromEndpoints();
 }
 
-void WiFiService::RemoveEndpoint(const WiFiEndpointConstRefPtr endpoint) {
+void WiFiService::RemoveEndpoint(const WiFiEndpointConstRefPtr &endpoint) {
   set<WiFiEndpointConstRefPtr>::iterator i = endpoints_.find(endpoint);
   DCHECK(i != endpoints_.end());
   if (i == endpoints_.end()) {