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

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

Issue 2709573006: MD Settings: Internet page cleanup + simple tests (Closed)
Patch Set: Rebase + additional cleanup Created 3 years, 10 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 11 matching lines...) Expand all
22 value: chrome.networkingPrivate, 22 value: chrome.networkingPrivate,
23 }, 23 },
24 24
25 /** Preferences state. */ 25 /** Preferences state. */
26 prefs: { 26 prefs: {
27 type: Object, 27 type: Object,
28 notify: true, 28 notify: true,
29 }, 29 },
30 30
31 /** 31 /**
32 * The device state for each network device type. Set by network-summary.
33 * @type {!Object<chrome.networkingPrivate.DeviceStateProperties>|undefined}
34 * @private
35 */
36 deviceStates: {
37 type: Object,
38 notify: true,
39 },
40
41 /**
42 * Highest priority connected network or null. Set by network-summary.
43 * @type {?CrOnc.NetworkStateProperties|undefined}
44 */
45 defaultNetwork: {
46 type: Object,
47 notify: true,
48 },
49
50 /**
32 * The network type for the known networks subpage. 51 * The network type for the known networks subpage.
33 * @private 52 * @private
34 */ 53 */
35 knownNetworksType_: String, 54 knownNetworksType_: String,
36 55
37 /** 56 /**
38 * Whether the 'Add connection' section is expanded. 57 * Whether the 'Add connection' section is expanded.
39 * @private 58 * @private
40 */ 59 */
41 addConnectionExpanded_: { 60 addConnectionExpanded_: {
(...skipping 10 matching lines...) Expand all
52 * @private 71 * @private
53 */ 72 */
54 thirdPartyVpnProviders_: { 73 thirdPartyVpnProviders_: {
55 type: Array, 74 type: Array,
56 value: function() { 75 value: function() {
57 return []; 76 return [];
58 } 77 }
59 }, 78 },
60 }, 79 },
61 80
81 // Element event listeners
82 listeners: {
83 'device-enabled-toggled': 'onDeviceEnabledToggled_',
84 'show-detail': 'onShowDetail_',
85 'show-known-networks': 'onShowKnownNetworks_',
86 },
87
88 // chrome.networkingPrivate listeners
89 /** @private {Function} */
90 onExtensionAddedListener_: null,
91
92 /** @private {Function} */
93 onExtensionRemovedListener_: null,
94
95 /** @private {Function} */
96 onExtensionDisabledListener_: null,
97
62 /** @override */ 98 /** @override */
63 attached: function() { 99 attached: function() {
64 this.boundOnExtensionAdded_ = this.boundOnExtensionAdded_ || 100 this.onExtensionAddedListener_ = this.onExtensionAddedListener_ ||
65 this.onExtensionAdded_.bind(this); 101 this.onExtensionAdded_.bind(this);
66 chrome.management.onInstalled.addListener(this.boundOnExtensionAdded_); 102 chrome.management.onInstalled.addListener(this.onExtensionAddedListener_);
67 chrome.management.onEnabled.addListener(this.boundOnExtensionAdded_); 103 chrome.management.onEnabled.addListener(this.onExtensionAddedListener_);
68 104
69 this.boundOnExtensionRemoved_ = this.boundOnExtensionRemoved_ || 105 this.onExtensionRemovedListener_ = this.onExtensionRemovedListener_ ||
70 this.onExtensionRemoved_.bind(this); 106 this.onExtensionRemoved_.bind(this);
71 chrome.management.onUninstalled.addListener( 107 chrome.management.onUninstalled.addListener(
72 this.boundOnExtensionRemoved_); 108 this.onExtensionRemovedListener_);
73 109
74 this.boundOnExtensionDisabled_ = this.boundOnExtensionDisabled_ || 110 this.onExtensionDisabledListener_ = this.onExtensionDisabledListener_ ||
75 this.onExtensionDisabled_.bind(this); 111 this.onExtensionDisabled_.bind(this);
76 chrome.management.onDisabled.addListener(this.boundOnExtensionDisabled_); 112 chrome.management.onDisabled.addListener(this.onExtensionDisabledListener_);
77 113
78 chrome.management.getAll(this.onGetAllExtensions_.bind(this)); 114 chrome.management.getAll(this.onGetAllExtensions_.bind(this));
79 115
80 this.networkingPrivate.getGlobalPolicy(function(policy) { 116 this.networkingPrivate.getGlobalPolicy(function(policy) {
81 this.globalPolicy_ = policy; 117 this.globalPolicy_ = policy;
82 }.bind(this)); 118 }.bind(this));
83 }, 119 },
84 120
85 /** @override */ 121 /** @override */
86 detached: function() { 122 detached: function() {
87 chrome.management.onInstalled.removeListener( 123 chrome.management.onInstalled.removeListener(
88 assert(this.boundOnExtensionAdded_)); 124 assert(this.onExtensionAddedListener_));
89 chrome.management.onEnabled.removeListener( 125 chrome.management.onEnabled.removeListener(
90 assert(this.boundOnExtensionAdded_)); 126 assert(this.onExtensionAddedListener_));
91 chrome.management.onUninstalled.removeListener( 127 chrome.management.onUninstalled.removeListener(
92 assert(this.boundOnExtensionRemoved_)); 128 assert(this.onExtensionRemovedListener_));
93 chrome.management.onDisabled.removeListener( 129 chrome.management.onDisabled.removeListener(
94 assert(this.boundOnExtensionDisabled_)); 130 assert(this.onExtensionDisabledListener_));
95 }, 131 },
96 132
97 /** 133 /**
98 * Reference to the bound listener, such that it can be removed on detach. 134 * Event triggered by a device state enabled toggle.
99 * @private {Function} 135 * @param {!{detail: {enabled: boolean,
136 * type: chrome.networkingPrivate.NetworkType}}} event
137 * @private
100 */ 138 */
101 boundOnExtensionAdded_: null, 139 onDeviceEnabledToggled_: function(event) {
102 140 if (event.detail.enabled)
103 /** 141 this.networkingPrivate.enableNetworkType(event.detail.type);
104 * Reference to the bound listener, such that it can be removed on detach. 142 else
105 * @private {Function} 143 this.networkingPrivate.disableNetworkType(event.detail.type);
106 */ 144 },
107 boundOnExtensionRemoved_: null,
108
109 /**
110 * Reference to the bound listener, such that it can be removed on detach.
111 * @private {Function}
112 */
113 boundOnExtensionDisabled_: null,
114 145
115 /** 146 /**
116 * @param {!{detail: !CrOnc.NetworkStateProperties}} event 147 * @param {!{detail: !CrOnc.NetworkStateProperties}} event
117 * @private 148 * @private
118 */ 149 */
119 onShowDetail_: function(event) { 150 onShowDetail_: function(event) {
120 var params = new URLSearchParams; 151 var params = new URLSearchParams;
121 params.append('guid', event.detail.GUID); 152 params.append('guid', event.detail.GUID);
122 params.append('type', event.detail.Type); 153 params.append('type', event.detail.Type);
123 if (event.detail.Name) 154 if (event.detail.Name)
(...skipping 120 matching lines...) Expand 10 before | Expand all | Expand 10 after
244 }, 275 },
245 276
246 /** 277 /**
247 * @param {!chrome.networkingPrivate.ThirdPartyVPNProperties} provider 278 * @param {!chrome.networkingPrivate.ThirdPartyVPNProperties} provider
248 * @return {string} 279 * @return {string}
249 */ 280 */
250 getAddThirdParrtyVpnLabel_: function(provider) { 281 getAddThirdParrtyVpnLabel_: function(provider) {
251 return this.i18n('internetAddThirdPartyVPN', provider.ProviderName); 282 return this.i18n('internetAddThirdPartyVPN', provider.ProviderName);
252 } 283 }
253 }); 284 });
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698