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

Unified Diff: Source/devtools/front_end/SettingsScreen.js

Issue 170273003: DevTools: Implement extensions-based shortcut bindings (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebased patch Created 6 years, 9 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: Source/devtools/front_end/SettingsScreen.js
diff --git a/Source/devtools/front_end/SettingsScreen.js b/Source/devtools/front_end/SettingsScreen.js
index cd558beb909a9a53c0715120ecfcf67bb0c7410c..e237c9912da6782298db3eef39c4b424feafe7b9 100644
--- a/Source/devtools/front_end/SettingsScreen.js
+++ b/Source/devtools/front_end/SettingsScreen.js
@@ -722,6 +722,24 @@ WebInspector.SettingsController.prototype =
/**
* @constructor
+ * @implements {WebInspector.ActionDelegate}
+ */
+WebInspector.SettingsController.SettingsScreenActionDelegate = function() { }
+
+WebInspector.SettingsController.SettingsScreenActionDelegate.prototype = {
+ /**
+ * @param {!KeyboardEvent=} keyEvent
+ * @return {boolean}
+ */
+ handleAction: function(keyEvent)
+ {
+ WebInspector.settingsController.showSettingsScreen(!!keyEvent ? WebInspector.SettingsScreen.Tabs.General : undefined);
pfeldman 2014/03/24 16:54:22 How can there be no keyEvent?
apavlov 2014/03/25 10:03:45 That was a provisional version to handle both keyb
+ return true;
+ }
+}
+
+/**
+ * @constructor
* @extends {WebInspector.Object}
* @param {function(!Element, string, ?string)} itemRenderer
*/

Powered by Google App Engine
This is Rietveld 408576698