[gnome-shell/wip/overviewTransitions: 6/7] overviewControls: don't override explicit calls to slideIn

[Carlos Soriano <csoriano src gnome org>]

commit 2ab14c4359a1e780ef5e70e4650b6ac8669f04bd
Author: Carlos Soriano <carlos soriano89 gmail com>
Date:   Sat Jul 26 17:36:17 2014 +0200

    overviewControls: don't override explicit calls to slideIn
    
    Currently we are overriding the explicit calls to slideIn
    given that it's called also with the signal of showing overview.
    
    It was necesary because of the bug that previous patch fixed,
    so now we can just delete that.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=732901

 js/ui/overviewControls.js |   10 ++--------
 1 files changed, 2 insertions(+), 8 deletions(-)
---
diff --git a/js/ui/overviewControls.js b/js/ui/overviewControls.js
index b396ff0..261c103 100644
--- a/js/ui/overviewControls.js
+++ b/js/ui/overviewControls.js
@@ -118,7 +118,6 @@ const SlidingControl = new Lang.Class({
                                      style_class: 'overview-controls',
                                      clip_to_allocation: true });
 
-        Main.overview.connect('showing', Lang.bind(this, this._onOverviewShowing));
         Main.overview.connect('hiding', Lang.bind(this, this._onOverviewHiding));
 
         Main.overview.connect('item-drag-begin', Lang.bind(this, this._onDragBegin));
@@ -177,14 +176,9 @@ const SlidingControl = new Lang.Class({
                                         transition: 'easeOutQuad' });
     },
 
-    _onOverviewShowing: function() {
-        this._visible = true;
-        this.layout.slideX = this._getSlide();
-        this.layout.translationX = this._getTranslation();
-        this.slideIn();
-    },
-
     _onOverviewHiding: function() {
+        // We need to explicitily slideOut since showing pages
+        // doesn't implies to slide out, instead, hiding the overview does.
         this.slideOut();
     },