Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(66)

Side by Side Diff: chrome/browser/resources/settings/internet_page/internet_page.js

Issue 2651483002: MD Settings: Eliminate use of ES6 for Chrome OS (Closed)
Patch Set: . Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 /** 5 /**
6 * @fileoverview 6 * @fileoverview
7 * 'settings-internet-page' is the settings page containing internet 7 * 'settings-internet-page' is the settings page containing internet
8 * settings. 8 * settings.
9 */ 9 */
10 Polymer({ 10 Polymer({
(...skipping 141 matching lines...) Expand 10 before | Expand all | Expand 10 after
152 chrome.send('addNetwork', [CrOnc.Type.VPN]); 152 chrome.send('addNetwork', [CrOnc.Type.VPN]);
153 }, 153 },
154 154
155 /** 155 /**
156 * @param {!{model: 156 * @param {!{model:
157 * !{item: !chrome.networkingPrivate.ThirdPartyVPNProperties}, 157 * !{item: !chrome.networkingPrivate.ThirdPartyVPNProperties},
158 * }} event 158 * }} event
159 * @private 159 * @private
160 */ 160 */
161 onAddThirdPartyVpnTap_: function(event) { 161 onAddThirdPartyVpnTap_: function(event) {
162 let provider = event.model.item; 162 var provider = event.model.item;
163 chrome.send('addNetwork', [CrOnc.Type.VPN, provider.ExtensionID]); 163 chrome.send('addNetwork', [CrOnc.Type.VPN, provider.ExtensionID]);
164 }, 164 },
165 165
166 /** 166 /**
167 * chrome.management.getAll callback. 167 * chrome.management.getAll callback.
168 * @param {!Array<!chrome.management.ExtensionInfo>} extensions 168 * @param {!Array<!chrome.management.ExtensionInfo>} extensions
169 * @private 169 * @private
170 */ 170 */
171 onGetAllExtensions_: function(extensions) { 171 onGetAllExtensions_: function(extensions) {
172 let vpnProviders = []; 172 var vpnProviders = [];
173 for (var extension of extensions) 173 for (var e = 0; e < extensions.length; ++e)
dpapad 2017/01/23 19:12:54 Same (optional) nit as before here. |e| would make
stevenjb 2017/01/24 02:02:04 Done.
174 this.addVpnProvider_(vpnProviders, extension); 174 this.addVpnProvider_(vpnProviders, extensions[e]);
175 this.thirdPartyVpnProviders_ = vpnProviders; 175 this.thirdPartyVpnProviders_ = vpnProviders;
176 }, 176 },
177 177
178 /** 178 /**
179 * If |extension| is a third-party VPN provider, add it to |vpnProviders|. 179 * If |extension| is a third-party VPN provider, add it to |vpnProviders|.
180 * @param {!Array<!chrome.networkingPrivate.ThirdPartyVPNProperties>} 180 * @param {!Array<!chrome.networkingPrivate.ThirdPartyVPNProperties>}
181 * vpnProviders 181 * vpnProviders
182 * @param {!chrome.management.ExtensionInfo} extension 182 * @param {!chrome.management.ExtensionInfo} extension
183 * @private 183 * @private
184 */ 184 */
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
241 }, 241 },
242 242
243 /** 243 /**
244 * @param {!chrome.networkingPrivate.ThirdPartyVPNProperties} provider 244 * @param {!chrome.networkingPrivate.ThirdPartyVPNProperties} provider
245 * @return {string} 245 * @return {string}
246 */ 246 */
247 getAddThirdParrtyVpnLabel_: function(provider) { 247 getAddThirdParrtyVpnLabel_: function(provider) {
248 return this.i18n('internetAddThirdPartyVPN', provider.ProviderName); 248 return this.i18n('internetAddThirdPartyVPN', provider.ProviderName);
249 } 249 }
250 }); 250 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698