Fix for requiring delegate methods (cappuccino#36), and for JSONP not…

… calling the run loop. [cappuccino#36 state:resolved];
280north · Sep 18, 2008 · 53d4a71 · 53d4a71
1 parent a64d1a1
commit 53d4a71
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/AppKit/CPAnimation.j b/AppKit/CPAnimation.j
@@ -211,7 +211,7 @@ ACTUAL_FRAME_RATE = 0;
 - (void)startAnimation
 {
     // If we're already animating, or our delegate stops us, animate.
-    if (_timer || _delegate && ![_delegate animationShouldStart:self])
+    if (_timer || _delegate && [_delegate respondsToSelector:@selector(animationShouldStart)] && ![_delegate animationShouldStart:self])
         return;
 
     _progress = 0.0;
@@ -258,7 +258,8 @@ ACTUAL_FRAME_RATE = 0;
     window.clearTimeout(_timer);
     _timer = nil;
 
-    [_delegate animationDidStop:self];
+    if ([_delegate respondsToSelector:@selector(animationDidStop:)])
+        [_delegate animationDidStop:self];
 }
 
 /*

diff --git a/Foundation/CPJSONPConnection.j b/Foundation/CPJSONPConnection.j
@@ -56,6 +56,8 @@ CPJSONPConnectionCallbacks = {};
     {
         [_delegate connection:self didReceiveData:data];
         [self removeScriptTag];
+
+        [[CPRunLoop currentRunLoop] performSelectors];
     };
 
     if(shouldStartImmediately)