Implement networkingPrivate.getNetworks

chrome/common/extensions/api/networking_private.json

Index: chrome/common/extensions/api/networking_private.json
diff --git a/chrome/common/extensions/api/networking_private.json b/chrome/common/extensions/api/networking_private.json
index db603ccf76241100603be8fa2a5cd8c4010b39b6..dd3b9ac1644121231bf64f7dbaec35d488b28ed5 100644
--- a/chrome/common/extensions/api/networking_private.json
+++ b/chrome/common/extensions/api/networking_private.json
@@ -113,7 +113,7 @@
       },
       {
         "name": "getState",
-        "description": "Gets the cached read-only properties of the network with id networkGuid.  This is meant to be a higher performance function than getProperties, which requires a round trip to query the networking subsystem.  It only returns a subset of the properties returned by getProperties.",
+        "description": "Gets the cached read-only properties of the network with id networkGuid. This is meant to be a higher performance function than getProperties, which requires a round trip to query the networking subsystem.  It only returns the following properties: GUID, Type, Name, ConnectionState, ErrorState, WiFi.Security, WiFi.SignalStrength, Cellular.NetworkTechnology, Cellular.ActivationState, Cellular.RoamingState. Cellular.OutOfCredits",
         "parameters": [
           {
             "name": "networkGuid",
@@ -181,13 +181,58 @@
         ]
       },
       {
+        "name": "getNetworks",
+        "description": "Returns a list of network objects with the same properties provided by getState.",
+        "parameters": [
+          {
+            "name": "filter",
+            "type": "object",
+            "properties": {
+              "type": {
+		"type": "string",
+		"enum": ["All", "Bluetooth", "Cellular", "Ethernet", "VPN", "Wireless", "WiFi", "WiMax"],
+		"description": "The type of networks to return."
+              },
+	      "visible": {
+		"type": "boolean",
+		"optional": true,
+		"description": "If true, only include visible (physically connected or in-range) networks. Defaults to 'false'."

[tbarzic, 2014/05/13 19:02:58]

I'd lose '' around false

+              },
+              "configured": {
+		"type": "boolean",
+		"optional": true,
+		"description": "If true, only include configured (saved) networks. Defaults to 'false'."
+              },
+              "limit": {
+		"type": "integer",
+		"optional": true,
+		"description": "Maximum number of networks to return. Default: 1000"
+              }
+	    }
+	  },
+          {
+            "name": "callback",
+            "type": "function",
+            "optional": true,
+            "parameters": [
+              {
+                "name": "networkList",
+                "type": "array",
+                "items": { "$ref": "NetworkProperties" }
+              }
+            ]
+          }
+        ]
+      },
+      {
         "name": "getVisibleNetworks",
-        "description": "Returns a list of visible network objects with the following ONC properties: GUID, Type, Name, ConnectionState, ErrorState, WiFi.Security, WiFi.SignalStrength, Cellular.NetworkTechnology, Cellular.ActivationState, Cellular.RoamingState. Cellular.OutOfCredits",
+	"deprecated": "Please Use $(ref:networkingPrivate.getNetworks).",

[tbarzic, 2014/05/13 19:02:58]

replace leading /t with ws

+        "description": "Returns a list of visible network objects with the same properties provided by getState.",
         "parameters": [
           {
             "name": "type",
             "type": "string",
-            "enum": ["Ethernet", "WiFi", "Bluetooth", "Cellular", "VPN", "All"],
+            "enum": ["All", "Bluetooth", "Cellular", "Ethernet", "VPN", "Wireless", "WiFi", "WiMax"],
             "description": "The type of networks to return."
           },
           {