From f8abe073eb5e531fb796ed016735498a78e90886 Mon Sep 17 00:00:00 2001 From: Carlos Garnacho Date: Wed, 14 Feb 2018 12:46:48 +0100 Subject: [PATCH] keyboard: Remove needless function It boils down to another function call, and has a single caller. Just let the caller make _setActiveLayer() directly. https://gitlab.gnome.org/GNOME/gnome-shell/issues/46 --- js/ui/keyboard.js | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/js/ui/keyboard.js b/js/ui/keyboard.js index 270166997..ad973685d 100644 --- a/js/ui/keyboard.js +++ b/js/ui/keyboard.js @@ -813,7 +813,7 @@ var Keyboard = new Lang.Class({ })); extraButton.connect('released', Lang.bind(this, function() { if (switchToLevel != null) - this._onLevelChanged(switchToLevel); + this._setActiveLayer(switchToLevel); else if (keyval != null) this._keyboardController.keyvalRelease(keyval); else if (action == 'hide') @@ -902,10 +902,6 @@ var Keyboard = new Lang.Class({ this.actor.height = maxHeight; }, - _onLevelChanged: function (level) { - this._setActiveLayer(level); - }, - _onGroupChanged: function () { this._ensureKeysForGroup(this._keyboardController.getCurrentGroup()); this._setActiveLayer(0);