[gnome-shell/wip/carlosg/fix-issue-46: 2/8] keyboard: Remove needless function



commit f8abe073eb5e531fb796ed016735498a78e90886
Author: Carlos Garnacho <carlosg gnome org>
Date:   Wed Feb 14 12:46:48 2018 +0100

    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);


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]