sproutcore 1.10.0.rc.3 → 1.10.0

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    ZmUxOTcxMGViYzE0OWVlYmJiMzlhNTUwMzk1M2I5ZDM1NzFiYWMzOA==
+    YmFiZjRlOTRmZjI1ZWY4NGI4ZGVjYjNmZjgyY2MzNzY2ODlmMWFhMg==
   data.tar.gz: !binary |-
-    M2VkMDZmODFjODExNDE4ZDY1NzQzYzVlNTU3NzdlZmE0MTU3NmQzMw==
+    MTk5ODA0ZmRiZmRmZjkwYzViZGY3OGIyNjgwMTIxMzYwMWE4NDEyZg==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    OWM1OGJhNDk5OWFiNmVlOWM4YjZjMzk3OWExYzU1NjliODUxMDZmZjYxMjUz
-    NDE2ODZiN2M5ZGQ3NzdmYTg1YWU5NjEwY2NkMWYzYjdkNDcyM2MyZGY5ODg3
-    NDEwZGFmMjMyMGQzMTNkMGFiNDMyYzBhNDBlNWQwYzAxZWFiZGY=
+    OGY5N2UwMWM5YmQ5ZTExOTJiYjViYjY3YWRkMWE3ZmU3ZGJhNzBhNGY2N2Vl
+    OTlhOWVkMGFhMzBhNjFiY2EyZTExMGQ5MzdlMzIxYjBhZWZiNTI3MDY1YjAy
+    ZmNlYzI0YzhjOTFjNWNkNmI1YWZmY2U2MDY5Y2MyMjMwMWJlMzI=
   data.tar.gz: !binary |-
-    NjYxYzdhZmYyMDI4N2FiN2UyZDJkNTlhNTBkMjJmYmM5MjI4ZmJkNDg3OTdh
-    ZjhhYzcwNzg0MzgzOGI4M2FjNGY4NzNhZTBiOTA2YTJkMjA0Y2YzOTMxOTE4
-    ZmQxMTU2ZDg0YzI3NjkyMTFjMDFkZGFkOGM0M2Q3NjU2NTMyOGQ=
+    OTM5NjA3NThiYTcwNjUyODA4Yjc4YThiNmM0ZjEwYmI2NDk0ZDVlZDI4MmEy
+    YTkzMGM3NWRlNWIzNWFhOTMwM2MyMmM2NzYxZDNkZWZlZTlkYWIxZGU3Zjk1
+    NWRhNGZjMTdmMDcwMWM1ZmEzMThjNDEwYmYzMGZkMTNjMWEwMjU=

data/VERSION.yml

@@ -1,4 +1,4 @@
 ---
 :major: 1
 :minor: 10
-:patch: 0.rc.3
+:patch: 0

data/lib/buildtasks/manifest.rake

@@ -96,15 +96,16 @@ namespace :manifest do
     manifest = env[:manifest]
 
     # these directories are to be excluded unless CONFIG.load_"dirname" = true
-    dirnames = %w(debug tests fixtures protocols).reject do |k|
+    exclude_dirnames = %w(debug tests fixtures protocols).reject do |k|
       CONFIG[:"load_#{k}"]
     end
+    exclude_dirnames.concat [*CONFIG[:exclude]]
 
     # loop through entries and hide those that do not below...
     manifest.entries.each do |entry|
 
       # if in /dirname or /foo.lproj/dirname -- hide it!
-      dirnames.each do |dirname|
+      exclude_dirnames.each do |dirname|
         if entry[:filename] =~ /^(([^\/]+)\.lproj\/)?#{dirname}\/.+$/
           entry.hide!
           next

data/lib/frameworks/sproutcore/Buildfile

@@ -48,7 +48,9 @@ config :designer,        :required => [:runtime, :foundation, :desktop, :templat
 config :"experimental/split_view", :test_required => [:desktop]
 
 # WRAPPER FRAMEWORKS
-config :sproutcore, :required => [:desktop, :datastore, :statechart]
+config :sproutcore,
+  :required => [:desktop, :datastore, :statechart],
+  :exclude => ['phantomjs']
 
 config :qunit, :required => []
 config :testing, :required => [:jquery], :test_required => [], :debug_required => []

data/lib/frameworks/sproutcore/CHANGELOG.md

@@ -1,8 +1,32 @@
 CHANGE LOG
 ==========
 
-1.10.0
-----------
+1.10.0.0
+-----------
+
+### CHANGES & FEATURES
+
+* Allows adjust to be called after animate in the same run loop. Adjust can occur cleanly, which won't affect the animation or it can clash, in which case it will override the animation.
+* Improves internal code structure to support optimization by JS engines.
+* Makes the conditional that attempts to lock all textfields from receiving focus behind a modal pane, fail more quickly (this also prevents getting the pane on views that may not yet have a pane).
+
+### DEPRECATIONS & REMOVALS
+
+* Removes 26.6MB of design files from within the framework. This means that these files are no longer downloaded and duplicated in each clone of the framework nor are any of them included in the gem (although many were stripped out of the gem anyhow). These files are still available at https://github.com/sproutcore/design.
+
+### BUG FIXES
+
+* Removes the blockers that prevented all browsers that support touch events from using mouse events. Note: a browser's support of touch events is no indication of a touch capable device or even that the touch capable device will not need to send mouse events (via a plugged in mouse for example). When implementing touch support in a custom control, you should call evt.preventDefault in touchEnd to prevent additional mouse events from being sent by the browser.
+* Prevents extremely bizarre bug where an iPad will fail to detect the os in iOS, in spite of following the correct code path. The correct value is assigned to an array and then re-read from the same array a moment later. For some unknown reason, when the value is re-read from the array it would return the old value. Side effects of this are that all hacks looking for mobile safari would fail (including touch handling in text fields).
+* Fixes a regression in SC.ScrollView that threw an exception when beginning a new touch drag while a previous one was still decelerating.
+* Prevents successive clicks with two different mouse buttons from being considered a double click. 
+* Fixes a bad conditional that coerced 0 frame widths to a false value. This caused excessive reloads of SC.GridView.
+* Fixes #1093 issue where a view would fail to be hidden if its pane was removed mid-transition.
+* Fixes #1087 issue with triggering and canceling animation in same runloop.
+* Fixes bug where PanelPane's ModalPane would appear if isModal becomes true while the PanelPane is transitioning out.
+
+1.10.0.rc.1, 1.10.0.rc.2, 1.10.0.rc.3
+-----------
 
 ### CHANGES & FEATURES
 

data/lib/frameworks/sproutcore/apps/showcase/resources/main_page.js

@@ -33,6 +33,9 @@ Showcase.mainPage = SC.Page.design({
           classNames: ['main-source-list'],
           contentBinding: SC.Binding.oneWay('Showcase.sourceTreeController.arrangedObjects'),
           contentValueKey: 'name',
+
+          // Show all the items for best performance since the list is short.
+          contentIndexesInRect: function () { return null; }.property().cacheable(),
           selectionBinding: SC.Binding.from('Showcase.sourceTreeController.selection')
         })
       }),

data/lib/frameworks/sproutcore/apps/showcase/views/views_item_view.js

@@ -60,7 +60,7 @@ Showcase.ViewsItemView = SC.View.extend({
       newContent.awake();
 
       sc_super();
-    },
+    }
   }),
 
   supportButton: SC.ButtonView.design({

data/lib/frameworks/sproutcore/apps/welcome/english.lproj/main_page.js

@@ -42,6 +42,9 @@ Welcome.mainPage = SC.Page.design({
             contentIconKey: "targetIcon",
             hasContentIcon: YES,
 
+            // Show all the items for best performance since the list is short.
+            contentIndexesInRect: function () { return null; }.property().cacheable(),
+
             target: "Welcome.targetsController",
             action: "loadApplication"
           })

data/lib/frameworks/sproutcore/frameworks/bootstrap/system/browser.js

@@ -129,7 +129,7 @@ SC.detectBrowser = function (userAgent, language) {
       conExp = '(?:[\\/:\\::\\s:;])', // Match the connecting character
       numExp = '(\\S+[^\\s:;:\\)]|)', // Match the "number"
       nameAndVersion,
-      osAndVersion,
+      os, osAndVersion,
       override;
 
   // Use the current values if none are provided.
@@ -265,9 +265,9 @@ SC.detectBrowser = function (userAgent, language) {
     [null, SC.BROWSER.unknown, '0'];
 
   // Normalize the os name.
-  if (isIOSDevice) { osAndVersion[1] = SC.OS.ios; }
-  else if (osAndVersion[1] === 'mac os x' || osAndVersion[1] === 'mac os') { osAndVersion[1] = SC.OS.mac; }
-  else if (osAndVersion[1] === 'windows nt') { osAndVersion[1] = SC.OS.win; }
+  if (isIOSDevice) { os = SC.OS.ios; }
+  else if (osAndVersion[1] === 'mac os x' || osAndVersion[1] === 'mac os') { os = SC.OS.mac; }
+  else if (osAndVersion[1] === 'windows nt') { os = SC.OS.win; }
 
   // Normalize the os version.
   osAndVersion[2] = osAndVersion[2] ? osAndVersion[2].replace(/_/g, '.') : '0';
@@ -277,7 +277,7 @@ SC.detectBrowser = function (userAgent, language) {
     @name SC.browser.os
     @type SC.OS|SC.BROWSER.unknown
   */
-  browser.os = osAndVersion[1];
+  browser.os = os;
 
   /**
     @name SC.browser.osVersion

data/lib/frameworks/sproutcore/frameworks/bootstrap/tests/system/browser.js

@@ -445,6 +445,15 @@ var userAgents = {
     engine: SC.ENGINE.webkit,
     engineVersion: '533.17.9'
   },
+  "Mozilla/5.0 (iPad; CPU OS 7_0 like Mac OS X) AppleWebKit/537.51.1 (KHTML, like Gecko) Version/7.0 Mobile/11A465 Safari/9537.53": {
+    device: SC.DEVICE.ipad,
+    name: SC.BROWSER.safari,
+    version: '7.0',
+    os: SC.OS.ios,
+    osVersion: '7.0',
+    engine: SC.ENGINE.webkit,
+    engineVersion: '537.51.1'
+  },
 
 // Android
 

data/lib/frameworks/sproutcore/frameworks/core_foundation/panes/pane_statechart.js

@@ -12,7 +12,7 @@ sc_require("panes/pane");
 SC.Pane.reopen({
 
   /** @private */
-  _notifyAttached: function () {
+  _notifyDidAttach: function () {
     // hook into root responder
     var responder = (this.rootResponder = SC.RootResponder.responder);
     responder.panes.add(this);
@@ -39,7 +39,7 @@ SC.Pane.reopen({
   },
 
   /** @private */
-  _notifyDetached: function () {
+  _notifyWillDetach: function () {
     sc_super();
 
     // Legacy.

data/lib/frameworks/sproutcore/frameworks/core_foundation/system/browser.js

@@ -9,6 +9,12 @@
 SC.mixin(SC.browser,
 /** @scope SC.browser */ {
 
+  /* @private Internal property for the cache of pre-determined experimental names. */
+  _cachedNames: null,
+
+  /* @private Internal property for the test element used for style testing. */
+  _testEl: null,
+
   /** @private */
   _testSupportFor: function (target, propertyName, testValue) {
     /*jshint eqnull:true*/

data/lib/frameworks/sproutcore/frameworks/core_foundation/system/root_responder.js

@@ -627,11 +627,8 @@ SC.RootResponder = SC.Object.extend(
     @returns {void}
   */
   setup: function () {
-    // handle touch events
-    this.listenFor(['touchstart', 'touchmove', 'touchend', 'touchcancel'], document);
-
     // handle basic events
-    this.listenFor(['keydown', 'keyup', 'beforedeactivate', 'mousedown', 'mouseup', 'click', 'dblclick', 'mousemove', 'selectstart', 'contextmenu'], document)
+    this.listenFor(['touchstart', 'touchmove', 'touchend', 'touchcancel', 'keydown', 'keyup', 'beforedeactivate', 'mousedown', 'mouseup', 'click', 'dblclick', 'mousemove', 'selectstart', 'contextmenu'], document)
         .listenFor(['resize'], window);
 
     if (SC.browser.isIE8OrLower) this.listenFor(['focusin', 'focusout'], document);
@@ -1272,7 +1269,6 @@ SC.RootResponder = SC.Object.extend(
     // As a workaround just let the browser to use the default behavior.
     if (this.ignoreTouchHandle(evt)) return YES;
 
-
     var hidingTouchIntercept = NO;
 
     SC.run(function () {
@@ -1772,17 +1768,11 @@ SC.RootResponder = SC.Object.extend(
   //
 
   mousedown: function (evt) {
-    if (SC.platform.touch) {
-      evt.allowDefault();
-      this._lastMouseDownCustomHandling = YES;
-      return YES;
-    }
-
     // First, save the click count. The click count resets if the mouse down
     // event occurs more than 250 ms later than the mouse up event or more
-    // than 8 pixels away from the mouse down event.
+    // than 8 pixels away from the mouse down event or if the button used is different.
     this._clickCount += 1;
-    if (!this._lastMouseUpAt || ((Date.now() - this._lastMouseUpAt) > 250)) {
+    if (!this._lastMouseUpAt || this._lastClickWhich !== evt.which || ((Date.now() - this._lastMouseUpAt) > 250)) {
       this._clickCount = 1;
     } else {
       var deltaX = this._lastMouseDownX - evt.clientX,
@@ -1793,6 +1783,7 @@ SC.RootResponder = SC.Object.extend(
     }
     evt.clickCount = this._clickCount;
 
+    this._lastClickWhich = evt.which;
     this._lastMouseDownX = evt.clientX;
     this._lastMouseDownY = evt.clientY;
 
@@ -1822,12 +1813,6 @@ SC.RootResponder = SC.Object.extend(
       dragView = this._drag,
       handler = null;
 
-    if (SC.platform.touch) {
-      evt.allowDefault();
-      this._lastMouseUpCustomHandling = YES;
-      return YES;
-    }
-
     if (dragView) {
       SC.run(function () {
         dragView.tryToPerform('mouseUp', evt);
@@ -1937,10 +1922,6 @@ SC.RootResponder = SC.Object.extend(
    trigger calls to mouseDragged.
   */
   mousemove: function (evt) {
-    if (SC.platform.touch) {
-      evt.allowDefault();
-      return YES;
-    }
 
     if (SC.browser.isIE) {
       if (this._lastMoveX === evt.clientX && this._lastMoveY === evt.clientY) return;
@@ -2051,54 +2032,35 @@ SC.RootResponder = SC.Object.extend(
   // ..........................................................
   // ANIMATION HANDLING
   //
+
+  /* @private Handler for animationstart events. */
   animationstart: function (evt) {
-    var view;
-    try {
-      view = this.targetViewForEvent(evt);
-      this.sendEvent('animationDidStart', evt, view);
-    } catch (e) {
-      SC.Logger.warn('Exception during animationDidStart: %@'.fmt(e));
-      throw e;
-    }
+    var view = this.targetViewForEvent(evt);
+    this.sendEvent('animationDidStart', evt, view);
 
     return view ? evt.hasCustomEventHandling : YES;
   },
 
+  /* @private Handler for animationiteration events. */
   animationiteration: function (evt) {
-    var view;
-    try {
-      view = this.targetViewForEvent(evt);
-      this.sendEvent('animationDidIterate', evt, view);
-    } catch (e) {
-      SC.Logger.warn('Exception during animationDidIterate: %@'.fmt(e));
-      throw e;
-    }
+    var view = this.targetViewForEvent(evt);
+    this.sendEvent('animationDidIterate', evt, view);
 
     return view ? evt.hasCustomEventHandling : YES;
   },
 
+  /* @private Handler for animationend events. */
   animationend: function (evt) {
-    var view;
-    try {
-      view = this.targetViewForEvent(evt);
-      this.sendEvent('animationDidEnd', evt, view);
-    } catch (e) {
-      SC.Logger.warn('Exception during animationDidEnd: %@'.fmt(e));
-      throw e;
-    }
+    var view = this.targetViewForEvent(evt);
+    this.sendEvent('animationDidEnd', evt, view);
 
     return view ? evt.hasCustomEventHandling : YES;
   },
 
+  /* @private Handler for transitionend events. */
   transitionend: function (evt) {
-    var view;
-    try {
-      view = this.targetViewForEvent(evt);
-      this.sendEvent('transitionDidEnd', evt, view);
-    } catch (e) {
-      SC.Logger.warn('Exception during transitionDidEnd: %@'.fmt(e));
-      throw e;
-    }
+    var view = this.targetViewForEvent(evt);
+    this.sendEvent('transitionDidEnd', evt, view);
 
     return view ? evt.hasCustomEventHandling : YES;
   }

data/lib/frameworks/sproutcore/frameworks/core_foundation/tests/views/view/animation.js

@@ -64,6 +64,63 @@ if (SC.platform.supportsCSSTransitions) {
     }, 5);
   });
 
+  test("animate + adjust: no conflict", function () {
+    stop(1000);
+
+    SC.run(function () {
+      view.animate('left', 100, { duration: 0.1 });
+      view.adjust('top', 100);
+      view.adjust({ 'width': null, 'right': 100 });
+    });
+
+    setTimeout(function () {
+      equals(view.get('layout').left, 100, 'left is');
+      equals(view.get('layout').top, 100, 'top is');
+      equals(view.get('layout').right, 100, 'right is');
+      equals(view.get('layout').width, undefined, 'width is');
+
+      SC.run(function () {
+        view.animate('top', 200, { duration: 0.1 });
+        view.adjust('left', 0);
+        view.adjust({ 'width': 100, 'right': null });
+      });
+
+      setTimeout(function () {
+        equals(view.get('layout').left, 0, 'left is');
+        equals(view.get('layout').top, 200, 'top is');
+        equals(view.get('layout').right, undefined, 'right is');
+        equals(view.get('layout').width, 100, 'width is');
+
+        start();
+      }, 200);
+    }, 200);
+  });
+
+  test("animate + adjust: conflict", function () {
+    stop(1000);
+
+    SC.run(function () {
+      view.animate('left', 100, { duration: 0.1 });
+      view.adjust('left', 200);
+    });
+
+    setTimeout(function () {
+      equals(view.get('layout').left, 200, 'left is');
+
+      SC.run(function () {
+        view.animate('top', 200, { duration: 0.1 });
+        // Adjust back to current value should still cancel the animation.
+        view.adjust('top', 0);
+      });
+
+      setTimeout(function () {
+        equals(view.get('layout').top, 0, 'top is');
+
+        start();
+      }, 200);
+    }, 200);
+  });
+
   test("callbacks work in general", function () {
     stop(2000);
 

data/lib/frameworks/sproutcore/frameworks/core_foundation/tests/views/view/isVisible.js

@@ -95,3 +95,27 @@ test("a view with visibility can have a child view without visibility", function
 
   ok(!errored, "Inserting a pane containing a child with visibility that itself has a child without visibility does not cause an error");
 });
+
+// Test for issue #1093.
+test("a view whose pane is removed during an isVisible transition gets correctly hidden", function() {
+  SC.RunLoop.begin();
+  var pane = SC.Pane.create({
+    childViews: ['childView'],
+    childView: SC.View.extend({
+      transitionHide: { run: function (view) {
+        view.animate('opacity', 0, 0.4, function () { this.didTransitionOut(); });
+      }}
+    })
+  });
+  pane.append();
+  pane.childView.set('isVisible', NO);
+  equals(pane.childView.get('viewState'), SC.CoreView.ATTACHED_HIDING, 'View is transitioning');
+  pane.remove();
+  SC.RunLoop.end();
+  SC.RunLoop.begin();
+  pane.append();
+  ok(pane.childView.$().hasClass('sc-hidden'), 'View was successfully hidden.')
+  pane.remove();
+  pane.destroy();
+  SC.RunLoop.end();
+});

data/lib/frameworks/sproutcore/frameworks/core_foundation/tests/views/view/view_states_test.js

@@ -46,6 +46,10 @@ test("Test unrendered state.", function () {
   // _doUpdateLayout()
 
   // UNHANDLED ACTIONS
+  handled = view._doShow();
+  ok(!handled, "Calling _doShow() should not be handled");
+  equals(view.viewState, SC.CoreView.UNRENDERED, "Calling _doShow() doesn't change state");
+
   handled = view._doAttach(document.body);
   ok(!handled, "Calling _doAttach(document.body) should not be handled");
   equals(view.viewState, SC.CoreView.UNRENDERED, "Calling _doAttach(document.body) doesn't change state");
@@ -58,7 +62,9 @@ test("Test unrendered state.", function () {
   ok(!handled, "Calling _doDetach() should not be handled");
   equals(view.viewState, SC.CoreView.UNRENDERED, "Calling _doDetach() doesn't change state");
 
-  handled = view._doHide();
+  SC.run(function () {
+    handled = view._doHide();
+  });
   ok(!handled, "Calling _doHide() should not be handled");
   equals(view.viewState, SC.CoreView.UNRENDERED, "Calling _doHide() doesn't change state");
 
@@ -73,10 +79,6 @@ test("Test unrendered state.", function () {
 
   // HANDLED ACTIONS
 
-  handled = view._doShow();
-  ok(handled, "Calling _doShow() should be handled");
-  equals(view.viewState, SC.CoreView.UNRENDERED, "Calling _doShow() doesn't change state");
-
   handled = view._doRender();
   ok(handled, "Calling _doRender() should be handled");
   equals(view.viewState, SC.CoreView.UNATTACHED, "Calling _doRender() changes state");
@@ -121,7 +123,9 @@ test("Test unattached state.", function () {
 
   // HANDLED ACTIONS
 
-  handled = view._doHide();
+  SC.run(function () {
+    handled = view._doHide();
+  });
   ok(handled, "Calling _doHide() should be handled");
   equals(view.viewState, SC.CoreView.UNATTACHED, "Calling _doHide() doesn't change state");
 
@@ -233,7 +237,9 @@ test("Test attached_shown state.", function () {
   view._doRender();
   view._doAttach(document.body);
 
-  handled = view._doHide();
+  SC.run(function () {
+    handled = view._doHide();
+  });
   ok(handled, "Calling _doHide() should be handled");
   equals(view.viewState, SC.CoreView.ATTACHED_HIDDEN, "Calling _doHide() changes state");
 
@@ -267,7 +273,7 @@ test("Calling destroy layer, clears the layer from all child views.",  function
 module("SC.View Adoption", {
 
   setup: function () {
-    parentView = SC.View.create();
+    parentView = SC.Pane.create();
   },
 
   teardown: function () {
@@ -364,7 +370,9 @@ test("Test showing and hiding parentView updates child views.", function () {
   ok(view.get('isVisibleInWindow'), "isVisibleInWindow should be true");
 
   // Hide the parentView.
-  parentView._doHide();
+  SC.run(function () {
+    parentView._doHide();
+  });
   equals(parentView.viewState, SC.CoreView.ATTACHED_HIDDEN, "A hidden parentView that is attached should be in the state");
   equals(view.viewState, SC.CoreView.ATTACHED_HIDDEN_BY_PARENT, "A child view of attached_hidden parentView should be in the state");
   equals(child.viewState, SC.CoreView.ATTACHED_HIDDEN_BY_PARENT, "A child view of attached_hidden parentView's child view should be in the state");
@@ -375,13 +383,18 @@ test("Test showing and hiding parentView updates child views.", function () {
   ok(handled, "Calling _doShow() on parentView should be handled");
   equals(view.viewState, SC.CoreView.ATTACHED_SHOWN, "Calling _doShow() on parentView changes state on view.");
   equals(child.viewState, SC.CoreView.ATTACHED_SHOWN, "Calling _doShow() on parentView changes state on child");
-  handled = view._doHide();
+
+  SC.run(function () {
+    handled = view._doHide();
+  });
   ok(handled, "Calling _doHide() should be handled");
   equals(view.viewState, SC.CoreView.ATTACHED_HIDDEN, "Calling _doHide() on view changes state on view");
   equals(child.viewState, SC.CoreView.ATTACHED_HIDDEN_BY_PARENT, "Calling _doHide() on view changes state on child");
 
   // Reset
-  parentView._doHide();
+  SC.run(function () {
+    parentView._doHide();
+  });
   view.destroy();
   child = SC.View.create();
   view = SC.View.create({ childViews: [child] });
@@ -423,11 +436,16 @@ test("Test hiding with transitionHide", function () {
 
   // Hide the parentView with transitionHide
   parentView.set('transitionHide', transitionHide);
-  parentView._doHide();
+  SC.run(function () {
+    parentView._doHide();
+  });
   ok(parentView.get('isVisibleInWindow'), "isVisibleInWindow of parentView should be false");
   ok(view.get('isVisibleInWindow'), "isVisibleInWindow should be true");
   ok(child.get('isVisibleInWindow'), "isVisibleInWindow of child should be true");
-  parentView.didTransitionOut(transitionHide);
+
+  SC.run(function () {
+    parentView.didTransitionOut();
+  });
   ok(!parentView.get('isVisibleInWindow'), "isVisibleInWindow of parentView should be false after didTransitionOut");
   ok(!view.get('isVisibleInWindow'), "isVisibleInWindow should be false after didTransitionOut");
   ok(!child.get('isVisibleInWindow'), "isVisibleInWindow of child should be false after didTransitionOut");