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

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

Issue 2984843003: MD Settings: Convert all usages of .bind(this) to use ES6 arrow function. (Closed)
Patch Set: Resolve conflicts. Created 3 years, 5 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 Polymer element for displaying and editing network proxy 6 * @fileoverview Polymer element for displaying and editing network proxy
7 * values. 7 * values.
8 */ 8 */
9 Polymer({ 9 Polymer({
10 is: 'network-proxy', 10 is: 'network-proxy',
(...skipping 199 matching lines...) Expand 10 before | Expand all | Expand 10 after
210 210
211 this.setProxyAsync_(proxy); 211 this.setProxyAsync_(proxy);
212 }, 212 },
213 213
214 /** 214 /**
215 * @param {!CrOnc.ProxySettings} proxy 215 * @param {!CrOnc.ProxySettings} proxy
216 * @private 216 * @private
217 */ 217 */
218 setProxyAsync_: function(proxy) { 218 setProxyAsync_: function(proxy) {
219 // Set this.proxy_ after dom-repeat has been stamped. 219 // Set this.proxy_ after dom-repeat has been stamped.
220 this.async(function() { 220 this.async(() => {
221 this.proxy_ = proxy; 221 this.proxy_ = proxy;
222 this.proxyModified_ = false; 222 this.proxyModified_ = false;
223 }.bind(this)); 223 });
224 }, 224 },
225 225
226 /** @private */ 226 /** @private */
227 useSameProxyChanged_: function() { 227 useSameProxyChanged_: function() {
228 this.proxyModified_ = true; 228 this.proxyModified_ = true;
229 }, 229 },
230 230
231 /** @private */ 231 /** @private */
232 useSharedProxiesChanged_: function() { 232 useSharedProxiesChanged_: function() {
233 var pref = this.getPref('settings.use_shared_proxies'); 233 var pref = this.getPref('settings.use_shared_proxies');
(...skipping 254 matching lines...) Expand 10 before | Expand all | Expand 10 after
488 /** @type {!SettingsCheckboxElement} */ (this.$.allowShared) 488 /** @type {!SettingsCheckboxElement} */ (this.$.allowShared)
489 .resetToPrefValue(); 489 .resetToPrefValue();
490 this.$.confirmAllowSharedDialog.close(); 490 this.$.confirmAllowSharedDialog.close();
491 }, 491 },
492 492
493 /** @private */ 493 /** @private */
494 onAllowSharedDialogClose_: function() { 494 onAllowSharedDialogClose_: function() {
495 cr.ui.focusWithoutInk(assert(this.$$('#allowShared'))); 495 cr.ui.focusWithoutInk(assert(this.$$('#allowShared')));
496 }, 496 },
497 }); 497 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698