VPN: Disable autoconnect by default

[email protected]

(cherry picked from commit b556c19)

Bug: 803497
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I376899c280da4e017c729ebf462e9f30f24fb022
Reviewed-on: https://chromium-review.googlesource.com/877220
Commit-Queue: Steven Bennetts <[email protected]>
Reviewed-by: Kevin Cernekee <[email protected]>
Reviewed-by: Toni Barzic <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#530904}
Reviewed-on: https://chromium-review.googlesource.com/882549
Reviewed-by: Steven Bennetts <[email protected]>
Cr-Commit-Position: refs/branch-heads/3325@{#50}
Cr-Branched-From: bc084a8-refs/heads/master@{#530369}

Author: stevenjb

ui/webui/resources/cr_components/chromeos/network/network_config.js

@@ -365,12 +365,13 @@ Polymer({
 
     var propertiesToSet = this.getPropertiesToSet_();
     if (!this.guid || this.getSource_() == CrOnc.Source.NONE) {
-      // New network configurations default to 'AutoConnect' unless prohibited
-      // by policy.
+      // New non VPN network configurations default to 'AutoConnect' unless
+      // prohibited by policy.
+      var prohibitAutoConnect = this.globalPolicy &&
+          this.globalPolicy.AllowOnlyPolicyNetworksToConnect;
       CrOnc.setTypeProperty(
           propertiesToSet, 'AutoConnect',
-          !(this.globalPolicy &&
-            this.globalPolicy.AllowOnlyPolicyNetworksToConnect));
+          this.type != CrOnc.Type.VPN && !prohibitAutoConnect);
 
       // Create the configuration, then connect to it in the callback.
       this.networkingPrivate.createNetwork(