MD Settings: Eliminate use of ES6 for Chrome OS

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

Index: chrome/browser/resources/settings/internet_page/internet_page.js
diff --git a/chrome/browser/resources/settings/internet_page/internet_page.js b/chrome/browser/resources/settings/internet_page/internet_page.js
index b08722e71b87d47fe44fe8b6cadd8b6e358e9743..b23914580f2f6181250766a9bdba6f3f93d9bb3d 100644
--- a/chrome/browser/resources/settings/internet_page/internet_page.js
+++ b/chrome/browser/resources/settings/internet_page/internet_page.js
@@ -159,7 +159,7 @@ Polymer({
    * @private
    */
   onAddThirdPartyVpnTap_: function(event) {
-    let provider = event.model.item;
+    var provider = event.model.item;
     chrome.send('addNetwork', [CrOnc.Type.VPN, provider.ExtensionID]);
   },
 
@@ -169,9 +169,9 @@ Polymer({
    * @private
    */
   onGetAllExtensions_: function(extensions) {
-    let vpnProviders = [];
-    for (var extension of extensions)
-      this.addVpnProvider_(vpnProviders, extension);
+    var vpnProviders = [];
+    for (var e = 0; e < extensions.length; ++e)

[dpapad, 2017/01/23 19:12:54]

Same (optional) nit as before here. |e| would make more sense if you were doing
the following

extensions.forEach(function(e) {
  this.addVpnProvider_(vpnProviders, e);
}.bind(this));

such that |e| holds an item from the array instead of an index.

[stevenjb, 2017/01/24 02:02:04]

Done.

+      this.addVpnProvider_(vpnProviders, extensions[e]);
     this.thirdPartyVpnProviders_ = vpnProviders;
   },