Don't assume responder was passed in makeFirstResponder

robinou · May 24, 2010 · a989bf1 · a989bf1
1 parent 28b2d7f
commit a989bf1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/.gitignore b/.gitignore
@@ -5,3 +5,4 @@
 .gitattributes
 *.swp
 *.swo
+tmp/
diff --git a/frameworks/foundation/mixins/responder_context.js b/frameworks/foundation/mixins/responder_context.js
@@ -139,7 +139,7 @@ SC.ResponderContext = {
       console.log('%@: makeFirstResponder => %@'.fmt(this, this.responderNameFor(responder)));
     }
 
-    responder.set("becomingFirstResponder", YES);
+    if (responder) responder.set("becomingFirstResponder", YES);
 
     this._locked = YES;
     this._pendingResponder = null;
@@ -178,7 +178,7 @@ SC.ResponderContext = {
       this._pendingResponder = null;
     }
 
-    responder.set("becomingFirstResponder", NO);
+    if (responder) responder.set("becomingFirstResponder", NO);
 
     return this ;
   },