Fix FirstNetworkByType, and some minor debugging

The change to combine NetworkState and FavoriteState broke
FirstNetworkByType for VPN since it doesn't appear with the "visible"
networks. We don't use that method a lot, so the easy fix is to remove
the early exit and continue instead of break.

BUG=375955
TBR=pneubeck@chromium.org

Review URL: https://codereview.chromium.org/332713003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@276851 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/resources/chromeos/network_ui/network_ui.html

@@ -39,6 +39,7 @@
       <td>Name</td>
       <td>Type</td>
       <td>State</td>
+      <td>Connect?</td>
       <td>Error</td>
       <td>Security</td>
       <td>Technology</td>

chrome/browser/resources/chromeos/network_ui/network_ui.js

@@ -13,6 +13,7 @@ var NetworkUI = (function() {
     'Name',
     'Type',
     'ConnectionState',
+    'connectable',
     'ErrorState',
     'WiFi.Security',
     ['Cellular.NetworkTechnology',

chromeos/network/network_state_handler.cc

@@ -222,7 +222,7 @@ const NetworkState* NetworkStateHandler::FirstNetworkByType(
     if (!network->update_received())
       continue;
     if (!network->visible())
-      break;
+      continue;
     if (network->Matches(type))
       return network;
   }

chromeos/network/network_util.cc

@@ -170,6 +170,7 @@ scoped_ptr<base::ListValue> TranslateNetworkListToONC(
         TranslateNetworkStateToONC(*it);
 
     if (debugging_properties) {
+      onc_dictionary->SetBoolean("connectable", (*it)->connectable());
       onc_dictionary->SetBoolean("visible", (*it)->visible());
       onc_dictionary->SetString("profile_path", (*it)->profile_path());
       std::string onc_source = (*it)->ui_data().GetONCSourceAsString();