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

Unified Diff: chrome/browser/resources/network_menu.js

Issue 3678006: Implement action interface in network_menu.js for 'connect'. (Closed)
Patch Set: Set status for non connected networks. Created 10 years, 2 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: chrome/browser/resources/network_menu.js
diff --git a/chrome/browser/resources/network_menu.js b/chrome/browser/resources/network_menu.js
index d07e152997ad9f8a55bf567a7b654c57cafbe2cb..2f0e767117c6be561be9b16045ec9de1237e63a0 100644
--- a/chrome/browser/resources/network_menu.js
+++ b/chrome/browser/resources/network_menu.js
@@ -3,10 +3,10 @@
// found in the LICENSE file.
/**
- * Sends 'activate' DOMUI message.
+ * Sends "connect" using the 'action' DOMUI message.
*/
-function sendAction(values) {
- chrome.send('action', values);
+function sendConnect(index, passphrase, identity) {
+ chrome.send('action', [ 'connect', String(index), passphrase, identity ]);
}
var NetworkMenuItem = cr.ui.define('div');
@@ -74,4 +74,18 @@ NetworkMenu.prototype = {
return new MenuItem();
}
},
+
+ onKeydown_: function(event) {
+ switch (event.keyIdentifier) {
+ case 'Enter':
+ case 'U+0020': // space
+ // Temporary, for testing sendConnect()
+ sendConnect(this.getMenuItemIndexOf(this.current_),
+ "passphrase", "identity");
+ break;
+ default:
+ Menu.prototype.onKeydown_.call(this, event);
+ break;
+ }
+ }
};

Powered by Google App Engine
This is Rietveld 408576698