sproutcore 1.11.0.rc1 → 1.11.0.rc2

data/lib/frameworks/sproutcore/frameworks/desktop/tests/views/scroll/touch.js

@@ -79,8 +79,11 @@ module("SC.ScrollView touch", {
     evt.pageY = 50;
     evt.changedTouches = [evt];
   },
+
   teardown: function() {
     SC.run(pane.destroy, pane);
+
+    pane = scrollView = inner = targetLayer = evt = null;
   }
 });
 
@@ -89,25 +92,19 @@ module("SC.ScrollView touch", {
 // but since there was no scroll, it will give inner views a chance to respond to it on touchEnd.
 test("Tapping with delaysContentTouches: YES", function() {
   // Trigger touchstart
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchstart', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchstart', evt);
 
   equals(scrollStart, 1, "After touchstart, the scroll view's touchStart should have been called once");
   equals(innerStart, 0, "After touchstart, the inner view's touchStart should not have been called, as the touch was captured by the scroll view");
 
   // Trigger touchmove:
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchmove', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchmove', evt);
 
   equals(scrollDragged, 1, "After touchmove, the scroll view's touchesDragged should have been called once");
   equals(innerDragged, 0, "After touchmove, the inner view's touchesDragged should not have been called, as the touch is still owned by the scroll view");
 
   // Trigger touchend:
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchend', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchend', evt);
 
   equals(scrollEnd, 1, "After touchend, the scroll view's touchEnd should have been called once");
   equals(innerStart, 1, "Once the scroll view has handled touchEnd, it passes the touch to the inner view, so innerStart should have run");
@@ -120,9 +117,7 @@ test("Tapping with delaysContentTouches: YES", function() {
 // touch, and since there was a scroll, the inner view will not receive any notifications whatsoever.
 test("Dragging with delaysContentTouches: YES", function() {
   // Trigger touchstart
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchstart', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchstart', evt);
 
   equals(scrollStart, 1, "After touchstart, the scroll view's touchStart should have been called once");
   equals(innerStart, 0, "After touchstart, the inner view's touchStart should not have been called, as the touch was captured by the scroll view");
@@ -130,17 +125,13 @@ test("Dragging with delaysContentTouches: YES", function() {
   // Give the event some vertical delta:
   evt.pageY += 16;
   // Trigger touchmove:
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchmove', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchmove', evt);
 
   equals(scrollDragged, 1, "After touchmove, the scroll view's touchesDragged should have been called once");
   equals(innerDragged, 0, "After touchmove, the inner view's touchesDragged should not have been called, as the touch is still owned by the scroll view");
 
   // Trigger touchend:
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchend', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchend', evt);
 
   equals(scrollEnd, 1, "After touchend, the scroll view's touchEnd should have been called once");
   equals(innerStart, 0, "inner view's touchStart will not have run, as the touch has moved enough to begin scrolling and will bypass the inner view entirely");
@@ -154,25 +145,19 @@ test("Tapping with delaysContentTouches: NO", function() {
   scrollView.set('delaysContentTouches', NO);
 
   // Trigger touchstart
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchstart', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchstart', evt);
 
   equals(scrollStart, 0, "We're not capturing touches, so scroll view's touchStart will not have fired after touchstart");
   equals(innerStart, 1, "After touchstart, inner view's touchStart will have been called once");
 
   // Trigger touchmove:
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchmove', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchmove', evt);
 
   equals(scrollDragged, 0, "Scroll view's touchesDragged will not have fired, as it is not the touch responder");
   equals(innerDragged, 1, "After touchmove, inner view's touchDragged gets straightforwardly called because it is the touch responder");
 
   // Trigger touchend:
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchend', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchend', evt);
 
   equals(scrollEnd, 0, "Again, the scroll view is completely uninvolved in this touch, so its touchEnd doesn't get called");
   equals(innerEnd, 1, "The inner view's touchEnd gets called because of how it's responding to the touch");
@@ -186,9 +171,7 @@ test("Dragging with delaysContentTouches: NO", function() {
   scrollView.set('delaysContentTouches', NO);
 
   // Trigger touchstart
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchstart', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchstart', evt);
 
   equals(scrollStart, 0, "Since the scroll view isn't capturing touches, it gets no touchStart love");
   equals(innerStart, 1, "After touchstart, the inner view's touchStart should have been straightforwardly called");
@@ -196,9 +179,7 @@ test("Dragging with delaysContentTouches: NO", function() {
   // Give the event some vertical delta:
   evt.pageY += 16;
   // Trigger touchmove:
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchmove', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchmove', evt);
 
   equals(scrollDragged, 0, "The scroll view's touchDragged should not have been called, since at the time of the event it was not the touch's responder");
   equals(innerDragged, 1, "The inner view's touchesDragged should have straightforwardly handled the event");
@@ -206,9 +187,7 @@ test("Dragging with delaysContentTouches: NO", function() {
   equals(innerCancel, 1, "Having passed the touch back to the scroll view, the inner view's touchCancelled should have run");
 
   // Trigger touchend:
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchend', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchend', evt);
 
   equals(scrollEnd, 1, "After touchend, the scroll view's touchEnd should have been called once");
 });
@@ -259,13 +238,13 @@ module("SC.ScrollView touch thresholds and locks", {
 
 //   // Start touch
 //   SC.run(function() {
-//     SC.Event.trigger(targetLayer, 'touchstart', [evt]);
+//     SC.Event.trigger(targetLayer, 'touchstart', evt);
 //   });
 
 //   // Move touch up less than touchScrollThreshold.
 //   evt.pageY = initialPageY - 9;
 //   SC.run(function() {
-//     SC.Event.trigger(targetLayer, 'touchmove', [evt]);
+//     SC.Event.trigger(targetLayer, 'touchmove', evt);
 //   });
 //   equals(scrollView.get('verticalScrollOffset'), 0, "Scrolling less than touchScrollThreshold results in no scrolling");
 //   if (scrollView.get('horizontalScrollOffset') !== 0) ok(false, "A touch with no horizontal change shouldn't trigger a horizontal scroll!");
@@ -273,7 +252,7 @@ module("SC.ScrollView touch thresholds and locks", {
 //   // Move touch up more than touchScrollThreshold.
 //   evt.pageY = initialPageY - 11;
 //   SC.run(function() {
-//     SC.Event.trigger(targetLayer, 'touchmove', [evt]);
+//     SC.Event.trigger(targetLayer, 'touchmove', evt);
 //   });
 //   equals(scrollView.get('verticalScrollOffset'), 11, "Scrolling more than touchScrollThreshold results in scrolling");
 //   if (scrollView.get('horizontalScrollOffset') !== 0) ok(false, "A touch with no horizontal change shouldn't trigger a horizontal scroll!");
@@ -281,7 +260,7 @@ module("SC.ScrollView touch thresholds and locks", {
 //   // Move touch sideways less than touchSecondaryScrollThreshold.
 //   evt.pageX = initialPageX - 19;
 //   SC.run(function() {
-//     SC.Event.trigger(targetLayer, 'touchmove', [evt]);
+//     SC.Event.trigger(targetLayer, 'touchmove', evt);
 //   });
 //   if (scrollView.get('verticalScrollOffset') !== 11) ok(false, "A touch with no vertical change shouldn't trigger a vertical scroll!");
 //   equals(scrollView.get('horizontalScrollOffset'), 0, "With a vertical scroll in motion, scrolling horizontally less than touchSecondaryScrollThreshold results in no scrolling");
@@ -289,7 +268,7 @@ module("SC.ScrollView touch thresholds and locks", {
 //   // Move touch sideways more than touchSecondaryScrollThreshold.
 //   evt.pageX = initialPageX - 21;
 //   SC.run(function() {
-//     SC.Event.trigger(targetLayer, 'touchmove', [evt]);
+//     SC.Event.trigger(targetLayer, 'touchmove', evt);
 //   });
 //   if (scrollView.get('verticalScrollOffset') !== 11) ok(false, "A touch with no vertical change shouldn't trigger a vertical scroll!");
 //   equals(scrollView.get('horizontalScrollOffset'), 21, "With a vertical scroll in motion, scrolling horizontally by more than touchSecondaryScrollThreshold results in scrolling");
@@ -301,26 +280,21 @@ test("Touch scroll lock", function() {
   equals(scrollView.get('horizontalScrollOffset'), 0, "PRELIM: Horizontal offset starts at");
 
   // Start touch
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchstart', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchstart', evt);
 
   // Move touch up more than touchSecondaryScrollLock.
   evt.pageY = initialPageY - SC.SCROLL.SCROLL_LOCK_GESTURE_THRESHOLD;
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchmove', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchmove', evt);
+
   equals(scrollView.get('verticalScrollOffset'), SC.SCROLL.SCROLL_LOCK_GESTURE_THRESHOLD, "PRELIM: Scrolling more than touchScrollThreshold results in scrolling");
   equals(scrollView.get('horizontalScrollOffset'), 0, "A touch with no horizontal change shouldn't trigger a horizontal scroll!");
 
   // Move touch sideways.
   evt.pageX = initialPageX - 50;
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchmove', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchmove', evt);
+
   equals(scrollView.get('verticalScrollOffset'), SC.SCROLL.SCROLL_LOCK_GESTURE_THRESHOLD, "A touch with no vertical change shouldn't trigger a vertical scroll!");
   equals(scrollView.get('horizontalScrollOffset'), 0, "Having scrolled vertically past the scrollGestureSecondaryThreshold, horizontal touch movements are ignored");
-
 });
 
 module("SC.ScrollView touch scale", {
@@ -388,19 +362,16 @@ test("Basic touch scale", function() {
   // Start touches.
   evt.touches = [];
   evt.changedTouches = [evt, evt2];
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchstart', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchstart', evt);
+
   equals(SC.RootResponder.responder.touchesForView(scrollView).length, 2, "Two touches should result in two touches");
 
   // Pinch out to 2x.
-  evt.touches = [evt, evt2];
-  evt.changedTouches = [evt, evt2];
   evt.pageX = evt.pageY -= 100;
   evt2.pageX = evt2.pageY += 100;
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchmove', [evt]);
-  });
+  evt.touches = [evt, evt2];
+  // evt.changedTouches = [touch1, touch2];
+  SC.Event.trigger(targetLayer, 'touchmove', evt);
 
   // SC.ScrollView's touch-pinching depends heavily on SC.RootResponder#averagedTouchesForView. If these tests
   // are misbehaving, first verify that SC.RootResponder's touch tests are passing.
@@ -411,9 +382,7 @@ test("Basic touch scale", function() {
   // Move the gesture.
   evt.pageX = evt.pageY += 100;
   evt2.pageX = evt2.pageY += 100;
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchmove', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchmove', evt);
 
   equals(scrollView.get('scale'), 2, "A gesture change with no change in distance should not change scale");
   equals(scrollView.get('horizontalScrollOffset'), 400, "Gesture change in position by 100 should move offsets by 100");
@@ -422,9 +391,7 @@ test("Basic touch scale", function() {
   // Move and pinch (back to 1x) in the same gesture.
   evt.pageX = evt.pageY = 400;
   evt2.pageX = evt2.pageY = 600;
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchmove', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchmove', evt);
 
   equals(scrollView.get('scale'), 1, "A pinch + move gesture should change the scale");
   equals(scrollView.get('horizontalScrollOffset'), 0, "A pinch + move gesture should update the horizontal offset correctly");
@@ -443,45 +410,38 @@ test("Adding and removing touches (no scaling).", function() {
   // Start touches.
   evt.touches = [];
   evt.changedTouches = [evt, evt2];
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchstart', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchstart', evt);
+
   equals(SC.RootResponder.responder.touchesForView(scrollView).length, 2, "Two touches should result in two touches");
 
-  // Update the touch lists.
-  evt.touches = [evt, evt2];
-  evt.changedTouches = [evt, evt2];
-  // Move touches up in tandem.
   evt.pageY -= 100;
   evt2.pageY -= 100;
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchmove', [evt]);
-  });
+  evt.touches = [evt, evt2];
+  SC.Event.trigger(targetLayer, 'touchmove', evt);
+
   equals(SC.RootResponder.responder.touchesForView(scrollView).length, 2, "There should still be two touches");
   equals(scrollView.get('scale'), 1, "A two-touch gesture with no pinching should result in no scaling");
   equals(scrollView.get('horizontalScrollOffset'), 0, "A two-touch vertical scroll gesture should not scroll horizontally");
   equals(scrollView.get('verticalScrollOffset'), 100, "A two-touch vertical scroll gesture should successfully scroll vertically");
 
   // Add a third touch.
-  evt3.touches = [evt, evt2];
-  evt3.changedTouches = [evt3];
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchstart', [evt3]);
-  });
+  evt.touches = [evt, evt2];
+  evt.changedTouches = [evt3];
+  SC.Event.trigger(targetLayer, 'touchstart', evt);
+
   equals(SC.RootResponder.responder.touchesForView(scrollView).length, 3, "Adding a third touch should result in three touches");
   equals(scrollView.get('scale'), 1, "Adding a third touch should not impact scaling");
   equals(scrollView.get('horizontalScrollOffset'), 0, "Adding a third touch should not impact horizontal offset");
   equals(scrollView.get('verticalScrollOffset'), 100, "Adding a third touch should not impact vertical offset");
 
   // Move all three touches up in tandem.
-  evt.touches = [evt, evt2, evt3];
-  evt.changedTouches = [evt, evt2, evt3];
   evt.pageY -= 100;
   evt2.pageY -= 100;
   evt3.pageY -= 100;
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchmove', [evt]);
-  });
+  evt.touches = [evt, evt2, evt3];
+  evt.changedTouches = [evt, evt2, evt3];
+  SC.Event.trigger(targetLayer, 'touchmove', evt);
+
   equals(scrollView.get('scale'), 1, "A three-touch gesture with no pinching should result in no scaling");
   equals(scrollView.get('horizontalScrollOffset'), 0, "A now-three-touch vertical scroll gesture should not scroll horizontally");
   equals(scrollView.get('verticalScrollOffset'), 200, "A now-three-touch vertical scroll gesture should successfully scroll vertically");
@@ -495,18 +455,15 @@ test("Adding and removing touches while scaling.", function() {
   // Start touches.
   evt.touches = [];
   evt.changedTouches = [evt, evt2];
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchstart', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchstart', evt);
   equals(SC.RootResponder.responder.touchesForView(scrollView).length, 2, "Two touches should result in two touches");
 
   // Pinch out to 2x to begin scaling.
   evt.touches = [evt, evt2];
   evt.pageX = evt.pageY = 300;
   evt2.pageX = evt2.pageY = 700;
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchmove', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchmove', evt);
+
   equals(scrollView.get('scale'), 2, "A 2x pinch gesture should double the scroll's scale");
   equals(scrollView.get('horizontalScrollOffset'), 500, "A centered pinch gesture should move the horizontal offset by half the content view's change in width");
   equals(scrollView.get('verticalScrollOffset'), 500, "A centered pinch gesture should move the vertical offset by half the content view's change in height");
@@ -514,9 +471,8 @@ test("Adding and removing touches while scaling.", function() {
   // Remove our second touch.
   evt2.touches = [evt, evt2];
   evt2.changedTouches = [evt2];
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchend', [evt2]);
-  });
+  SC.Event.trigger(targetLayer, 'touchend', evt2);
+
   equals(SC.RootResponder.responder.touchesForView(scrollView).length, 1, "Removing one of two touches should leave one touch");
   equals(scrollView.get('scale'), 2, "Removing a touch shouldn't change scale");
   equals(scrollView.get('horizontalScrollOffset'), 500, "Removing a touch shouldn't change the horizontal offset");
@@ -527,9 +483,8 @@ test("Adding and removing touches while scaling.", function() {
   evt3.changedTouches = [evt3];
   evt3.pageX = 700;
   evt3.pageY = 700;
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchstart', [evt3]);
-  });
+  SC.Event.trigger(targetLayer, 'touchstart', evt3);
+
   equals(SC.RootResponder.responder.touchesForView(scrollView).length, 2, "Adding one touch to one touch should result in two touches");
   equals(scrollView.get('scale'), 2, "Adding a touch shouldn't change scale");
   equals(scrollView.get('horizontalScrollOffset'), 500, "Adding a touch shouldn't change the horizontal offset");
@@ -540,9 +495,8 @@ test("Adding and removing touches while scaling.", function() {
   evt.changedTouches = [evt, evt3];
   evt.pageX = evt.pageY = 400;
   evt3.pageX = evt3.pageY = 600;
-  SC.run(function() {
-    SC.Event.trigger(targetLayer, 'touchmove', [evt]);
-  });
+  SC.Event.trigger(targetLayer, 'touchmove', evt);
+
   equals(scrollView.get('scale'), 1, "Pinching back down by half should reverse doubling of scaling");
   equals(scrollView.get('horizontalScrollOffset'), 0, "Pinching back down by half should reverse horizontal offset change");
   equals(scrollView.get('verticalScrollOffset'), 0, "Pinching back down by half should reverse vertical offset change");

data/lib/frameworks/sproutcore/frameworks/desktop/tests/views/segmented/ui.js

@@ -605,5 +605,20 @@ var pane;
     ok(SC.none(segmentedView.getPath('layout.width')), "Having flexible layout prevents view from auto-resizing.");
   });
 
+  test("Check adjusting a segment views layout updates its frame", function() {
+    var segmentedView;
+
+    segmentedView = pane.view('3_empty');
+
+    equals(segmentedView.get('layout').height, 25, '3_empty.layout should have a height of 25');
+    equals(segmentedView.get('frame').height, 25, '3_empty.layout should have a height of 25');
+
+    // adjust the views height
+    segmentedView.adjust('height', 30);
+
+    equals(segmentedView.get('layout').height, 30, '3_empty.layout should have a height of 30');
+    equals(segmentedView.get('frame').height, 30, '3_empty.layout should have a height of 30');
+  })
+
 
 })();

data/lib/frameworks/sproutcore/frameworks/desktop/views/button.js

@@ -57,7 +57,7 @@ SC.HOLD_BEHAVIOR = 'hold';
   @extends SC.Control
   @since SproutCore 1.0
 */
-SC.ButtonView = SC.View.extend(SC.Control,
+SC.ButtonView = SC.View.extend(SC.ActionSupport, SC.Control,
 /** @scope SC.ButtonView.prototype */ {
 
   /**
@@ -71,6 +71,32 @@ SC.ButtonView = SC.View.extend(SC.Control,
     return NO;
   }.property('isEnabledInPane'),
 
+  /**
+    The name of the method to call when the button is pressed.
+
+    This property is used in conjunction with the `target` property to execute a method when a
+    regular button is pressed. If you do not set a target, then pressing the button will cause a
+    search of the responder chain for a view that implements the action named. If you do set a
+    target, then the button will only try to call the method on that target.
+
+    The action method of the target should implement the following signature:
+
+        action: function (sender) {
+          // Return value is ignored by SC.ButtonView.
+        }
+
+    Therefore, if a target needs to know which button called its action, it should look to the
+    `sender` argument.
+
+    *NOTE:* This property is not relevant when the button is used in toggle mode. Toggle mode only
+    modifies the `value` of the button without triggering actions.
+
+    @type String
+    @default null
+    @see SC.ActionSupport
+  */
+  action: null,
+
   /**
     @type Array
     @default ['sc-button-view']
@@ -93,6 +119,31 @@ SC.ButtonView = SC.View.extend(SC.Control,
    */
   escapeHTML: true,
 
+  /**
+    The target to invoke the action on when the button is pressed.
+
+    If you set this target, the action will be called on the target object directly when the button
+    is clicked.  If you leave this property set to `null`, then the responder chain will be
+    searched for a view that implements the action when the button is pressed.
+
+    The action method of the target should implement the following signature:
+
+        action: function (sender) {
+          // Return value is ignored by SC.ButtonView.
+        }
+
+    Therefore, if a target needs to know which button called its action, it should look to the
+    `sender` argument.
+
+    *NOTE:* This property is not relevant when the button is used in toggle mode. Toggle mode only
+    modifies the `value` of the button without triggering actions.
+
+    @type Object
+    @default null
+    @see SC.ActionSupport
+  */
+  target: null,
+
   /**
     The theme to apply to the button. By default, a subtheme with the name of
     'square' is created for backwards-compatibility.
@@ -299,41 +350,6 @@ SC.ButtonView = SC.View.extend(SC.Control,
   isCancel: NO,
   isCancelBindingDefault: SC.Binding.oneWay().bool(),
 
-  /**
-    The name of the action you want triggered when the button is pressed.
-
-    This property is used in conjunction with the target property to execute
-    a method when a regular button is pressed.  These properties are not
-    relevant when the button is used in toggle mode.
-
-    If you do not set a target, then pressing a button will cause the
-    responder chain to search for a view that implements the action you name
-    here.  If you set a target, then the button will try to call the method
-    on the target itself.
-
-    For legacy support, you can also set the action property to a function.
-    Doing so will cause the function itself to be called when the button is
-    clicked.  It is generally better to use the target/action approach and
-    to implement your code in a controller of some type.
-
-    @type String
-    @default null
-  */
-  action: null,
-
-  /**
-    The target object to invoke the action on when the button is pressed.
-
-    If you set this target, the action will be called on the target object
-    directly when the button is clicked.  If you leave this property set to
-    null, then the button will search the responder chain for a view that
-    implements the action when the button is pressed instead.
-
-    @type Object
-    @default null
-  */
-  target: null,
-
   /*
     TODO When is this property ever changed? Is this redundant with
     render delegates since it can now be turned on on a theme-by-theme
@@ -804,20 +820,16 @@ SC.ButtonView = SC.View.extend(SC.Control,
 
   /** @private */
   _runAction: function(evt) {
-    var action = this.get('action'),
-        target = this.get('target') || null,
-        rootResponder;
+    var action = this.get('action');
 
     if (action) {
+      // Legacy support for action functions.
       if (action && (SC.typeOf(action) === SC.T_FUNCTION)) {
         this.action(evt);
-        return;
+
+      // Use SC.ActionSupport.
       } else {
-        rootResponder = this.getPath('pane.rootResponder');
-        if (rootResponder) {
-          // newer action method + optional target syntax...
-          rootResponder.sendAction(action, target, this, this.get('pane'), null, this);
-        }
+        this.fireAction();
       }
     }
   },