[Networking] Fixed closure issues with networking struct validation

Fixes warnings produced when this CL is patched in
https://crrev.com/c/2006413 which does stricter type checking on mojom
structs from colsure.

Fixed: 1047844
Change-Id: Ia1dd0159329ec697bce6d5b832abc8a70d083706
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2050846
Commit-Queue: James Vecore <vecore@google.com>
Auto-Submit: James Vecore <vecore@google.com>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#740532}

chrome/browser/resources/settings/chromeos/internet_page/internet_detail_page.js

@@ -395,13 +395,37 @@ Polymer({
       return;
     }
 
+    const PolicySource = chromeos.networkConfig.mojom.PolicySource;
+
     let enforcement;
     let controlledBy;
-    if (autoConnect.enforced ||
-        (!!this.globalPolicy &&
-         !!this.globalPolicy.allowOnlyPolicyNetworksToAutoconnect)) {
+
+    if (this.globalPolicy &&
+        this.globalPolicy.allowOnlyPolicyNetworksToAutoconnect) {
+      enforcement = chrome.settingsPrivate.Enforcement.ENFORCED;
+      controlledBy = chrome.settingsPrivate.ControlledBy.DEVICE_POLICY;
+    } else {
+      switch (autoConnect.policySource) {
+        case PolicySource.kUserPolicyEnforced:
+        case PolicySource.kDevicePolicyEnforced:
           enforcement = chrome.settingsPrivate.Enforcement.ENFORCED;
+          break;
+        case PolicySource.kUserPolicyRecommended:
+        case PolicySource.kDevicePolicyRecommended:
+          enforcement = chrome.settingsPrivate.Enforcement.RECOMMENDED;
+          break;
+      }
+
+      switch (autoConnect.policySource) {
+        case PolicySource.kDevicePolicyEnforced:
+        case PolicySource.kDevicePolicyRecommended:
           controlledBy = chrome.settingsPrivate.ControlledBy.DEVICE_POLICY;
+          break;
+        case PolicySource.kUserPolicyEnforced:
+        case PolicySource.kUserPolicyRecommended:
+          controlledBy = chrome.settingsPrivate.ControlledBy.USER_POLICY;
+          break;
+      }
     }
 
     if (this.autoConnectPref_ &&
@@ -909,8 +933,7 @@ Polymer({
     if (!managedProperties) {
       return false;
     }
-    for (const key of Object.keys(managedProperties)) {
-      const value = managedProperties[key];
+    for (const value of Object.values(managedProperties)) {
       if (typeof value != 'object' || value === null) {
         continue;
       }

chrome/browser/resources/settings/chromeos/internet_page/internet_subpage.js

@@ -669,7 +669,7 @@ Polymer({
    * @private
    */
   matchesType_(typeString, device) {
-    return device &&
+    return !!device &&
         device.type == OncMojo.getNetworkTypeFromString(typeString);
   },