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

Unified Diff: ui/webui/resources/cr_elements/network/cr_network_select.js

Issue 2069323002: Add support for custom entries in <cr_network_select/>. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@604119--Implement-Chrome-OS-out-of-box-flow-in-Material-Design--ImplementNetworkSelectionScreen
Patch Set: Fix compile. Created 4 years, 6 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 side-by-side diff with in-line comments
Download patch
Index: ui/webui/resources/cr_elements/network/cr_network_select.js
diff --git a/ui/webui/resources/cr_elements/network/cr_network_select.js b/ui/webui/resources/cr_elements/network/cr_network_select.js
index ecf439385f56ff69a6631ef683e0f4062b8df36b..f6edde42bc5ccad70f2e0b861a777b8aec09f7de 100644
--- a/ui/webui/resources/cr_elements/network/cr_network_select.js
+++ b/ui/webui/resources/cr_elements/network/cr_network_select.js
@@ -56,12 +56,45 @@ Polymer({
/**
* List of all network state data for all visible networks.
+ * See <cr-network-list-network-item/> for details.
+ *
* @type {!Array<!CrOnc.NetworkStateProperties>}
*/
networkStateList: {
type: Array,
value: function() { return []; }
- }
+ },
+
+ /**
+ * List of custom items to display at the end of networks list.
+ * See <cr-network-list-custom-item/> for details.
+ *
+ * @type {!Array<Object>}
+ */
+ customItems: {
+ type: Array,
+ value: function() { return []; },
+ },
+
+ /**
+ * Show all buttons in list items.
+ */
+ showButtons: {
+ type: Boolean,
+ value: false,
+ },
+
+ /**
+ * Whether to handle "item-selected" for network items.
+ * If this property is false, "network-item-selected" event is fired
+ * carrying CrOnc.NetworkStateProperties as event detail.
+ *
+ * @type {Function}
+ */
+ handleNetworkItemSelected: {
+ type: Boolean,
+ value: false,
+ },
},
/**
@@ -102,7 +135,7 @@ Polymer({
},
/**
- * Polymer chnaged function.
+ * Polymer changed function.
* @private
*/
networkListOpenedChanged_: function() {
@@ -134,14 +167,21 @@ Polymer({
},
/**
- * Event triggered when a cr-network-list-item is selected.
+ * Event triggered when a cr-network-list-network-item is selected.
* @param {!{detail: !CrOnc.NetworkStateProperties}} event
* @private
*/
onNetworkListItemSelected_: function(event) {
var state = event.detail;
+
+ if (!this.handleNetworkItemSelected) {
+ this.fire("network-item-selected", state);
+ return;
+ }
+
if (state.ConnectionState != CrOnc.ConnectionState.NOT_CONNECTED)
return;
+
chrome.networkingPrivate.startConnect(state.GUID, function() {
var lastError = chrome.runtime.lastError;
if (lastError && lastError != 'connecting')
« no previous file with comments | « ui/webui/resources/cr_elements/network/cr_network_select.html ('k') | ui/webui/resources/cr_elements_resources.grdp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698