Fix VPN connected logic in internet settings

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

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @fileoverview
  * 'settings-internet-detail' is the settings subpage containing details
  * for a network.
  *
  * @group Chrome Settings Elements
@@ skipping 46 matching lines @@
 
     /**
      * The network IP Address.
      */
     IPAddress: {
       type: String,
       value: ''
     },
 
     /**
+     * Highest priority connected network or null.
+     * @type {?CrOnc.NetworkStateProperties}
+     */
+    defaultNetwork: {
+      type: Object,
+      value: null
+    },
+
+    /**
      * Object providing network type values for data binding.
      * @const
      */
     NetworkType: {
       type: Object,
       value: {
         CELLULAR: CrOnc.Type.CELLULAR,
         ETHERNET: CrOnc.Type.ETHERNET,
         VPN: CrOnc.Type.VPN,
         WIFI: CrOnc.Type.WI_FI,
@@ skipping 224 matching lines @@
         return false;
       }
       if (!properties.Cellular.MDN)
         return false;
     }
 
     return true;
   },
 
   /**
+   * @param {!CrOnc.NetworkProperties} properties
+   * @param {?CrOnc.NetworkStateProperties} defaultNetwork
    * @return {boolean} Whether or not to enable the network connect button.
    * @private
    */
-  enableConnect_: function(properties) {
+  enableConnect_: function(properties, defaultNetwork) {
     if (!properties || !this.showConnect_(properties))
       return false;
     if (properties.Type == CrOnc.Type.CELLULAR && CrOnc.isSimLocked(properties))
       return false;
-    // TODO(stevenjb): For VPN, check connected state of any network.
+    if (properties.Type == CrOnc.Type.VPN && !defaultNetwork)
+      return false;
     return true;
   },
 
   /**
    * @param {!CrOnc.NetworkProperties} properties
    * @return {boolean} Whether or not to show the 'Disconnect' button.
    * @private
    */
   showDisconnect_: function(properties) {
     return properties.Type != CrOnc.Type.ETHERNET &&
@@ skipping 340 matching lines @@
    */
   allPropertiesMatch_: function(curValue, newValue) {
     for (let key in newValue) {
       if (newValue[key] != curValue[key])
         return false;
     }
     return true;
   }
 });
 })();