Add getGlobalPolicy to networkingPrivate API.

extensions/common/api/networking_private.idl

Index: extensions/common/api/networking_private.idl
diff --git a/extensions/common/api/networking_private.idl b/extensions/common/api/networking_private.idl
index b1855c407e404c2b9063697a53a09fdac0e303e2..017966df74178fe168b56cb3cd4816c6b840604a 100644
--- a/extensions/common/api/networking_private.idl
+++ b/extensions/common/api/networking_private.idl
@@ -806,6 +806,15 @@ namespace networkingPrivate {
     long? limit;
   };
 
+  dictionary GlobalPolicy {
+    // If true, only allow policy networks to autoconnect. Defaults to false.

[tbarzic, 2017/01/06 22:43:18]

Since this describes state of the system, rather than a function arg, "only
allow" seems a bit misleading.
How about:
If true, only policy configured networks can auto-connect.

[stevenjb, 2017/01/09 19:30:37]

Done.

+    boolean? AllowOnlyPolicyNetworksToAutoconnect;
+
+    // If true, only allow policy networks to connect and no new networks can be
+    // added. Defaults to false.

[tbarzic, 2017/01/06 22:43:18]

I think "configured" would be clearer than "added".

[stevenjb, 2017/01/09 19:30:37]

Done.

+    boolean? AllowOnlyPolicyNetworksToConnect;
+  };
+
   callback VoidCallback = void();
   callback BooleanCallback = void(boolean result);
   callback StringCallback = void(DOMString result);
@@ -818,6 +827,7 @@ namespace networkingPrivate {
   callback GetDeviceStatesCallback = void(DeviceStateProperties[] result);
   callback GetEnabledNetworkTypesCallback = void(NetworkType[] result);
   callback CaptivePortalStatusCallback = void(CaptivePortalStatus result);
+  callback GetGlobalPolicyCallback = void(GlobalPolicy result);
 
   // These functions all report failures via chrome.runtime.lastError.
   interface Functions {
@@ -1034,6 +1044,10 @@ namespace networkingPrivate {
     static void setCellularSimState(DOMString networkGuid,
                                     CellularSimState simState,
                                     optional VoidCallback callback);
+
+    // Get the global policy properties. These properties are not expected to

[tbarzic, 2017/01/06 22:43:18]

suggestion:
s/Get/Gets/
s/are not expected to change during a session/are expected to be constant within
a session/

[stevenjb, 2017/01/09 19:30:37]

Fixed Gets. 'Expected to be constant' sounds more like a code comment than an
API comment to me. *shrug*

+    // change during a session.
+    static void getGlobalPolicy(GetGlobalPolicyCallback callback);
   };
 
   interface Events {