commit c8095687985d31c5c8d51e32e2f91efa6ba4ed5b
author Alex Vakulenko <avakulenko@google.com> Wed Aug 12 09:56:20 2015 -0700
committer Alex Vakulenko <avakulenko@google.com> Tue Aug 18 11:15:30 2015 -0700
tree 86c99de067db939d32e26c5c0e4101635867c407
parent 8940a5000452e1d657957bb89d8388cfc24f98ad

libweave: Fix wifi bootstrapping

We mistakenly go out of bootstrapping mode if any network change
is detected (even "transitions" like offline -> offline).

We only need to go from bootstrapping mode into monitoring when
network connection detects that there is network.

BUG=brillo:1233
TEST=`FEATURES=test emerge-link libweave`

Change-Id: I7afc715152fc3a2be5a53926c4e3b71c0d22cb79
Reviewed-on: https://chromium-review.googlesource.com/293023
Reviewed-by: Vitaly Buka <vitalybuka@chromium.org>
Tested-by: Alex Vakulenko <avakulenko@chromium.org>
Commit-Queue: Vitaly Buka <vitalybuka@chromium.org>
Tested-by: Vitaly Buka <vitalybuka@chromium.org>

libweave/src/privet/wifi_bootstrap_manager.cc

diff --git a/libweave/src/privet/wifi_bootstrap_manager.cc b/libweave/src/privet/wifi_bootstrap_manager.cc
index 84ceb3e..bea033f 100644
--- a/libweave/src/privet/wifi_bootstrap_manager.cc
+++ b/libweave/src/privet/wifi_bootstrap_manager.cc
@@ -232,7 +232,7 @@
   VLOG(3) << "ConnectivityChanged: " << is_connected;
   UpdateConnectionState();
 
-  if (state_ == State::kBootstrapping) {
+  if (state_ == State::kBootstrapping && is_connected) {
     StartMonitoring();
     return;
   }