Revert "Bug 1055299: Ignore continuous app launch request in homescre…

…en side and send homescreen to background while rocketbar is open in system side"

This reverts commit b866e8a.

apps/system/js/visibility_manager.js

@@ -1,4 +1,4 @@
-/* global attentionWindowManager, System, rocketbar */
+/* global attentionWindowManager, System */
 'use strict';
 
 (function(exports) {
@@ -88,8 +88,6 @@
         this.publish('showwindow', { type: evt.type });
         this._resetDeviceLockedTimer();
         break;
-
-      case 'rocketbar-overlayclosed':
       case 'lockscreen-request-unlock':
         var detail = evt.detail,
             activity = null,
@@ -100,8 +98,7 @@
           notificationId = detail.notificationId;
         }
 
-        if (!attentionWindowManager.hasActiveWindow() &&
-            !rocketbar.active) {
+        if (!attentionWindowManager.hasActiveWindow()) {
           this.publish('showwindow', {
             activity: activity,  // Trigger activity opening in AWM
             notificationId: notificationId
@@ -110,7 +107,6 @@
         this._resetDeviceLockedTimer();
         break;
       case 'lockscreen-appopened':
-      case 'rocketbar-overlayopened':
         // If the audio is active, the app should not set non-visible
         // otherwise it will be muted.
         // TODO: Remove this hack.
@@ -128,11 +124,13 @@
           this.publish('hidewindow', { type: evt.type });
         }
         break;
+      case 'rocketbar-overlayopened':
       case 'utility-tray-overlayopened':
       case 'cardviewshown':
       case 'system-dialog-show':
         this.publish('hidewindowforscreenreader');
         break;
+      case 'rocketbar-overlayclosed':
       case 'utility-tray-overlayclosed':
       case 'cardviewclosed':
       case 'system-dialog-hide':

apps/system/test/marionette/homescreen_navigation_test.js

@@ -64,7 +64,7 @@ marionette('Homescreen navigation >', function() {
 
     goHome();
     launchSettings();
-    client.helper.wait(1000);
+
     reflowHelper.startTracking(System.URL);
     client.switchToFrame();
 

apps/system/test/unit/visibility_manager_test.js

@@ -1,4 +1,4 @@
-/* globals MocksHelper, VisibilityManager, MockRocketbar,
+/* globals MocksHelper, VisibilityManager,
            MockAttentionWindowManager, MockTaskManager, MockAppWindow */
 'use strict';
 
@@ -8,7 +8,6 @@ requireApp('system/shared/test/unit/mocks/mock_manifest_helper.js');
 require('/shared/test/unit/mocks/mock_system.js');
 requireApp('system/test/unit/mock_attention_window_manager.js');
 requireApp('system/test/unit/mock_app_window.js');
-requireApp('system/test/unit/mock_rocketbar.js');
 
 var mocksForVisibilityManager = new MocksHelper([
   'AttentionWindowManager', 'System', 'AppWindow'
@@ -20,7 +19,6 @@ suite('system/VisibilityManager', function() {
   mocksForVisibilityManager.attachTestHelpers();
   setup(function(done) {
     window.attentionWindowManager = MockAttentionWindowManager;
-    window.rocketbar = new MockRocketbar();
     this.sinon.useFakeTimers();
 
     stubById = this.sinon.stub(document, 'getElementById');
@@ -38,51 +36,6 @@ suite('system/VisibilityManager', function() {
   });
 
   suite('handle events', function() {
-    test('rocketbar-overlayopened', function() {
-      visibilityManager._normalAudioChannelActive = false;
-      var stubPublish = this.sinon.stub(visibilityManager, 'publish');
-      visibilityManager.handleEvent({
-        type: 'rocketbar-overlayopened'
-      });
-
-      assert.isTrue(stubPublish.calledOnce);
-      assert.equal(stubPublish.getCall(0).args[0], 'hidewindow');
-
-      visibilityManager._normalAudioChannelActive = true;
-      visibilityManager.handleEvent({
-        type: 'rocketbar-overlayopened'
-      });
-
-      assert.isTrue(stubPublish.calledOnce);
-
-      visibilityManager._normalAudioChannelActive = false;
-    });
-
-    test('searchclosed', function() {
-      this.sinon.stub(MockAttentionWindowManager,
-        'hasActiveWindow').returns(false);
-      var stubPublish = this.sinon.stub(visibilityManager, 'publish');
-
-      visibilityManager.handleEvent({
-        type: 'rocketbar-overlayclosed'
-      });
-
-      assert.isTrue(stubPublish.calledOnce);
-      assert.isTrue(stubPublish.getCall(0).args[0] === 'showwindow');
-    });
-
-    test('searchclosed when there is active attention window', function() {
-      this.sinon.stub(MockAttentionWindowManager,
-        'hasActiveWindow').returns(true);
-      var stubPublish = this.sinon.stub(visibilityManager, 'publish');
-
-      visibilityManager.handleEvent({
-        type: 'searchclosed'
-      });
-
-      assert.isFalse(stubPublish.called);
-    });
-
     test('lock', function() {
       visibilityManager._normalAudioChannelActive = false;
       var stubPublish = this.sinon.stub(visibilityManager, 'publish');
@@ -157,6 +110,26 @@ suite('system/VisibilityManager', function() {
       assert.isTrue(stubPublish.calledWith('showlockscreenwindow'));
     });
 
+    test('rocketbar-overlayopened', function() {
+      var stubPublish = this.sinon.stub(visibilityManager, 'publish');
+      visibilityManager.handleEvent({
+        type: 'rocketbar-overlayopened'
+      });
+
+      assert.isTrue(stubPublish.called);
+      assert.isTrue(stubPublish.calledWith('hidewindowforscreenreader'));
+    });
+
+    test('rocketbar-overlayclosed', function() {
+      var stubPublish = this.sinon.stub(visibilityManager, 'publish');
+      visibilityManager.handleEvent({
+        type: 'rocketbar-overlayclosed'
+      });
+
+      assert.isTrue(stubPublish.called);
+      assert.isTrue(stubPublish.calledWith('showwindowforscreenreader'));
+    });
+
     test('utility-tray-overlayopened', function() {
       var stubPublish = this.sinon.stub(visibilityManager, 'publish');
       visibilityManager.handleEvent({

shared/elements/gaia_grid/js/grid_view.js

@@ -29,7 +29,6 @@
     this.onTouchEnd = this.onTouchEnd.bind(this);
     this.onScroll = this.onScroll.bind(this);
     this.onContextMenu = this.onContextMenu.bind(this);
-    this.onVisibilityChange = this.onVisibilityChange.bind(this);
     this.lastScrollTime = 0;
 
     if (config.features.zoom) {
@@ -74,11 +73,6 @@
       this.layout.cols = value;
     },
 
-    /**
-     * We are in the state of launching an app.
-     */
-    _launchingApp: false,
-
     /**
      * Adds an item into the items array.
      * If the item is an icon, add it to icons.
@@ -150,7 +144,6 @@
       this.element.addEventListener('touchend', this.onTouchEnd);
       this.element.addEventListener('contextmenu', this.onContextMenu);
       window.addEventListener('scroll', this.onScroll, true);
-      window.addEventListener('visibilitychange', this.onVisibilityChange);
       this.lastTouchStart = null;
     },
 
@@ -159,7 +152,6 @@
       this.element.removeEventListener('touchend', this.onTouchEnd);
       this.element.removeEventListener('contextmenu', this.onContextMenu);
       window.removeEventListener('scroll', this.onScroll, true);
-      window.removeEventListener('visibilitychange', this.onVisibilityChange);
       this.lastTouchStart = null;
     },
 
@@ -216,10 +208,6 @@
       }
     },
 
-    onVisibilityChange: function() {
-      this._launchingApp = false;
-    },
-
     /**
      * Launches an app.
      */
@@ -268,24 +256,6 @@
         }, returnTimeout);
       }
 
-      if ((icon.detail.type === 'app' || icon.detail.type === 'bookmark') &&
-          this._launchingApp) {
-        return;
-      }
-      if ((icon.detail.type === 'app' && icon.appState === 'ready') ||
-          icon.detail.type === 'bookmark') {
-        this._launchingApp = true;
-        if (this._launchingTimeout) {
-          window.clearTimeout(this._launchingTimeout);
-          this._launchingTimeout = null;
-        }
-        // This avoids some edge cases if we didn't get visibilitychange anyway.
-        this._launchingTimeout = window.setTimeout(function() {
-          this._launchingTimeout = null;
-          this._launchingApp = false;
-        }.bind(this), 3000);
-      }
-
       icon[action]();
     },