sproutcore 1.0.1009 → 1.0.1024

data/frameworks/sproutcore/frameworks/foundation/system/response.js

@@ -113,7 +113,9 @@ SC.Response = SC.Object.extend(
   }.property('request').cacheable(),
   
   /**
-    The response status code.  
+    The response status code.
+    
+    @property {Number}
   */
   status: -100, // READY
 
@@ -133,14 +135,24 @@ SC.Response = SC.Object.extend(
   encodedBody: null,
   
   /**
-    Response body.  If isJSON was set, will be parsed automatically.
+    Response body. If isJSON was set, will be parsed automatically.
     
-    @property {Hash|String}
+    @response {Hash|String|SC.Error} the response body or the parsed JSON.
+      Returns a SC.Error instance if there is a JSON parsing error.
   */
   body: function() {
     // TODO: support XML
     var ret = this.get('encodedBody');
-    if (ret && this.get('isJSON')) ret = SC.json.decode(ret);
+    if (ret && this.get('isJSON')) {
+      try {
+        ret = SC.json.decode(ret);
+      } catch(e) {
+        return SC.Error.create({
+          message: e.name + ': ' + e.message,
+          label: 'Response',
+          errorValue: this });
+      }
+    }
     return ret;
   }.property('encodedBody').cacheable(),
   
@@ -251,6 +263,10 @@ SC.Response = SC.Object.extend(
   */
   cancelTransport: function() {},
   
+  
+  /** @private
+    Will notify each listener.
+  */
   _notifyListener: function(listeners, status) {
     var info = listeners[status], params, target, action;
     if (!info) return NO ;
@@ -287,6 +303,9 @@ SC.Response = SC.Object.extend(
     return this ;
   },
   
+  /**
+    String representation of the response object
+  */
   toString: function() {
     var ret = sc_super();
     return "%@<%@ %@, status=%@".fmt(ret, this.get('type'), this.get('address'), this.get('status'));
@@ -385,7 +404,7 @@ SC.XHRResponse = SC.Response.extend({
         handleReadyStateChange = function() {
           if (!transport) return null ;
           var ret = transport.finishRequest();
-          transport = null ; // cleanup memory
+          if (ret) transport = null ; // cleanup memory
           return ret ;
         };
         rawRequest.onreadystatechange = handleReadyStateChange;
@@ -445,12 +464,11 @@ SC.XHRResponse = SC.Response.extend({
         this.set('status', status);
       
       }, this);
-    }
-     
-    if (readyState === 4) {
       // avoid memory leak in MSIE: clean up
       rawRequest.onreadystatechange = function() {} ;
+      return YES;
     }
+    return NO; 
   }
 
   

data/frameworks/sproutcore/frameworks/foundation/system/root_responder.js

@@ -149,7 +149,7 @@ SC.RootResponder = SC.Object.extend({
         // the top of the stack and make the specified pane the new keyPane.
         // First, though, do a sanity-check to make sure it's not already the
         // key pane, in which case we have nothing to do.
-        var previousKeyPane = this.get('keyPane') ;
+        previousKeyPane = this.get('keyPane') ;
         if (previousKeyPane === pane) {
           return this ;
         }
@@ -172,7 +172,7 @@ SC.RootResponder = SC.Object.extend({
       previousKeyPane = this.get('keyPane') ;
       previousKeyPanes = this.get('previousKeyPanes') ;
   
-      var newKeyPane = null ;
+      newKeyPane = null ;
       while (previousKeyPanes.length > 0) {
         var candidate = previousKeyPanes.pop();
         if (candidate.get('isPaneAttached')  &&  candidate.get('acceptsKeyPane')) {

data/frameworks/sproutcore/frameworks/foundation/system/timer.js

@@ -150,7 +150,7 @@ SC.Timer = SC.Object.extend(
     
     @property {Boolean}
   */
-  isPooled: YES,
+  isPooled: NO,
   
   /**
     The time interval in milliseconds.
@@ -408,7 +408,7 @@ SC.Timer = SC.Object.extend(
   /** @private - Default values to reset reused timers to. */
   RESET_DEFAULTS: {
     target: null, action: null, 
-    isPooled: YES, isPaused: NO, isScheduled: NO, isValid: YES,
+    isPooled: NO, isPaused: NO, isScheduled: NO, isValid: YES,
     interval: 0, repeats: NO, until: null,
     startTime: null, lastFireTime: 0
   },

data/frameworks/sproutcore/frameworks/foundation/system/utils.js

@@ -12,6 +12,12 @@ SC.mixin( /** @scope SC */ {
 
   _downloadFrames: 0, // count of download frames inserted into document
   
+  _copy_computed_props: [
+    "maxWidth", "maxHeight", "paddingLeft", "paddingRight", "paddingTop", "paddingBottom",
+    "fontFamily", "fontSize", "fontStyle", "fontWeight", "fontVariant", "lineHeight",
+    "whiteSpace"
+  ],
+  
   /**
     Starts a download of the file at the named path.
     
@@ -132,13 +138,13 @@ SC.mixin( /** @scope SC */ {
    */
   rectsEqual: function(r1, r2, delta) {
     if (!r1 || !r2) return (r1 == r2) ;
-    
-    if (delta === null) delta = 0.1;
+  
+    if (!delta && delta !== 0) delta = 0.1;
     if ((r1.y != r2.y) && (Math.abs(r1.y - r2.y) > delta)) return NO ; 
     if ((r1.x != r2.x) && (Math.abs(r1.x - r2.x) > delta)) return NO ; 
     if ((r1.width != r2.width) && (Math.abs(r1.width - r2.width) > delta)) return NO ; 
     if ((r1.height != r2.height) && (Math.abs(r1.height - r2.height) > delta)) return NO ; 
-    return true ;
+    return YES ;
   },
   
   /** Returns the insersection between two rectangles. 
@@ -269,19 +275,122 @@ SC.mixin( /** @scope SC */ {
     }
 
     style = '%@; width: %@px; left: %@px; position: absolute'.fmt(style, width, (-1*width));
-    elem.setAttribute('style', style);
+    SC.$(elem).attr('style', style);
 
     if (classes !== '') {
-      elem.setAttribute('class', classes);
+      SC.$(elem).attr('class', classes);
     }
 
-    elem.textContent = str;
+    elem.innerHTML = str;
     height = elem.clientHeight;
 
     elem = null; // don't leak memory
     return height;
   },
   
+  /**
+  Given a string and an example element or style string, and an optional
+  set of class names, calculates the width and height of that block of text.
+  
+  To constrain the width, set max-width on the exampleElement or in the style string.
+  
+  @param string {String} The string to measure.
+  @param exampleElement The example element to grab styles from, or the style string to use.
+  @param classNames {String} (Optional) Class names to add to the test element.
+  */
+  metricsForString: function(string, exampleElement, classNames)
+  {
+    var element = this._metricsCalculationElement, width, height, classes, styles, style;
+    
+    // collect the class names
+    classes = SC.A(classNames).join(' ');
+    
+    // get the calculation element
+    if (!element) {
+      element = this._metricsCalculationElement = document.createElement("div");
+      document.body.insertBefore(element, null);
+    }
+    
+    // two possibilities: example element or type string
+    if (SC.typeOf(exampleElement) != SC.T_STRING) {
+      var computed = null;
+      if (document.defaultView && document.defaultView.getComputedStyle) {
+        computed = document.defaultView.getComputedStyle(exampleElement, null);
+      } else {
+        computed = exampleElement.currentStyle;
+      }
+      
+      // set (lovely cssText property here helps a lot—if it works. Unfortunately, only Safari supplies it.)
+      style = computed.cssText;
+      
+      // if that didn't work (Safari-only?) go alternate route. This is SLOW code...
+      if (!style || style.trim() === "") {
+        // there is only one way to do it...
+        var props = this._copy_computed_props;
+        
+        // firefox ONLY allows this method
+        for (var i = 0; i < props.length; i++) {
+          var prop = props[i], val = computed[prop];
+          element.style[prop] = val;
+        }
+        
+        // and why does firefox specifically need "font" set?
+        var cs = element.style; // cached style
+        if (cs.font === "") {
+          var font = "";
+          if (cs.fontStyle) font += cs.fontStyle + " ";
+          if (cs.fontVariant) font += cs.fontVariant + " ";
+          if (cs.fontWeight) font += cs.fontWeight + " ";
+          if (cs.fontSize) font += cs.fontSize; else font += "10px"; //force a default
+          if (cs.lineHeight) font += "/" + cs.lineHeight;
+          font += " ";
+          if (cs.fontFamily) font += cs.fontFamily; else cs += "sans-serif";
+          
+          element.style.font = font;
+        }
+        
+        SC.mixin(element.style, {
+          left: "0px", top: "0px", position: "absolute", bottom: "auto", right: "auto", width: "auto", height: "auto"
+        });
+      }
+      else
+      {
+        // set style
+        element.setAttribute("style", style + "; position:absolute; left: 0px; top: 0px; bottom: auto; right: auto; width: auto; height: auto;");
+      }
+      
+      // clean up
+      computed = null;
+    } else {
+      // it is a style string already
+      style = exampleElement;
+      
+      // set style
+      element.setAttribute("style", style + "; position:absolute; left: 0px; top: 0px; bottom: auto; right: auto; width: auto; height: auto;");
+    }
+    
+    // the conclusion of which to use (innerText or textContent) should be cached
+    if (typeof element.innerText != "undefined") element.innerText = string;
+    else element.textContent = string;
+    
+    element.className = classes;
+    
+    // measure
+    var result = {
+      width: element.clientWidth,
+      height: element.clientHeight
+    };
+    
+    // clear element
+    element.innerHTML = "";
+    element.className = "";
+    element.setAttribute("style", ""); // get rid of any junk from computed style.
+    
+    // clean up
+    element = null;
+    return result;
+  },
+
   /** Finds the absolute viewportOffset for a given element.
     This method is more accurate than the version provided by prototype.
     
@@ -378,8 +487,8 @@ SC.mixin( /** @scope SC */ {
   /** Returns the union of two ranges.  If one range is null, the other
    range will be returned.  */
   unionRanges: function(r1, r2) { 
-    if ((r1 === null) || (r1.length < 0)) return r2 ;
-    if ((r2 === null) || (r2.length < 0)) return r1 ;
+    if ((r1 == null) || (r1.length < 0)) return r2 ;
+    if ((r2 == null) || (r2.length < 0)) return r1 ;
     
     var min = Math.min(r1.start, r2.start) ;
     var max = Math.max(SC.maxRange(r1), SC.maxRange(r2)) ;
@@ -388,7 +497,7 @@ SC.mixin( /** @scope SC */ {
   
   /** Returns the intersection of the two ranges or SC.RANGE_NOT_FOUND */
   intersectRanges: function(r1, r2) {
-    if ((r1 === null) || (r2 === null)) return SC.RANGE_NOT_FOUND ;
+    if ((r1 == null) || (r2 == null)) return SC.RANGE_NOT_FOUND ;
     if ((r1.length < 0) || (r2.length < 0)) return SC.RANGE_NOT_FOUND;
     var min = Math.max(SC.minRange(r1), SC.minRange(r2)) ;
     var max = Math.min(SC.maxRange(r1), SC.maxRange(r2)) ;
@@ -398,7 +507,7 @@ SC.mixin( /** @scope SC */ {
   
   /** Returns the difference of the two ranges or SC.RANGE_NOT_FOUND */
   subtractRanges: function(r1, r2) {
-    if ((r1 === null) || (r2 === null)) return SC.RANGE_NOT_FOUND ;
+    if ((r1 == null) || (r2 == null)) return SC.RANGE_NOT_FOUND ;
     if ((r1.length < 0) || (r2.length < 0)) return SC.RANGE_NOT_FOUND;
     var max = Math.max(SC.minRange(r1), SC.minRange(r2)) ;
     var min = Math.min(SC.maxRange(r1), SC.maxRange(r2)) ;
@@ -416,8 +525,8 @@ SC.mixin( /** @scope SC */ {
   */
   rangesEqual: function(r1, r2) {
     if (r1===r2) return true ;
-    if (r1 === null) return r2.length < 0 ;
-    if (r2 === null) return r1.length < 0 ;
+    if (r1 == null) return r2.length < 0 ;
+    if (r2 == null) return r1.length < 0 ;
     return (r1.start == r2.start) && (r1.length == r2.length) ;
   },
 
@@ -449,7 +558,7 @@ SC.mixin( /** @scope SC */ {
 
     var h = (max == min) ? 0 : ((max == rgb[0]) ? ((rgb[1]-rgb[2])/(max-min)/6) : ((max == rgb[1]) ? ((rgb[2]-rgb[0])/(max-min)/6+1/3) : ((rgb[0]-rgb[1])/(max-min)/6+2/3)));
     h = (h < 0) ? (h + 1) : ((h > 1)  ? (h - 1) : h);
-    var s = (max === 0) ? 0 : (1 - min/max);
+    var s = (max == 0) ? 0 : (1 - min/max);
     var v = max/255;
     return [h, s, v];
   },

data/frameworks/sproutcore/frameworks/foundation/tests/system/core_query/jquery_core.js

@@ -213,7 +213,6 @@ div#show-tests * { display: none; }\
 	<b id="floatTest">Float test.</b>\
 	<iframe id="iframe" name="iframe"></iframe>\
 	<form id="lengthtest">\
-		<input type="text" id="length" name="test"/>\
 		<input type="text" id="idTest" name="id"/>\
 	</form>\
 	<table id="table"></table>\
@@ -663,7 +662,7 @@ test("width()", function() {
  // IE is off by one on this.  We don't really care at this point since ems 
  // is not really central to most SC apps.
  $div.css("padding", "2em");
- var e = SC.browser.msie ? 29 : 30; 
+ var e = SC.browser.msie==8 ? 29 : 30; 
  equals($div.width(), e, "Test padding specified with ems");
 
  $div.css("border", "1em solid #fff");
@@ -697,7 +696,7 @@ test("height()", function() {
  // IE is off by one on this.  We don't really care at this point since ems 
  // is not really central to most SC apps.
  $div.css("padding", "2em");
- var e = SC.browser.msie ? 29 : 30; 
+ var e = SC.browser.msie==8 ? 29 : 30; 
  equals($div.height(), e, "Test padding specified with ems");
 
  $div.css("border", "1em solid #fff");

data/frameworks/sproutcore/frameworks/foundation/tests/system/render_context/tag.js

@@ -17,30 +17,30 @@ module("SC.RenderContext#tag", {
 
 test("should emit a self closing tag.  like calling begin().end()", function() {
   context.tag("input");
-  equals(context.get(1), "<input />");
+  equals(SC.RenderContext.escapeHTML(context.get(1)), SC.RenderContext.escapeHTML("<input />"));
 });
 
 test("should respect passed opts when emitting", function() {
   context.tag("foo") ;
   equals(context.length, 3);
-  equals(context.get(1), "<foo>");
-  equals(context.get(2), '<'+'/foo>');
+  equals(SC.RenderContext.escapeHTML(context.get(1)), SC.RenderContext.escapeHTML("<foo>"));
+  equals(SC.RenderContext.escapeHTML(context.get(2)), SC.RenderContext.escapeHTML('<'+'/foo>'));
 });
 
 test("should NOT emit self closing tag if tag is script", function() {
   context.tag("script");
-  equals(context.get(1), '<script>');
-  equals(context.get(2), '<'+'/script>');
+  equals(SC.RenderContext.escapeHTML(context.get(1)), SC.RenderContext.escapeHTML('<script>'));
+  equals(SC.RenderContext.escapeHTML(context.get(2)), SC.RenderContext.escapeHTML('<'+'/script>'));
 });
 
 test("should NOT emit self closing tag if tag is div", function() {
   context.tag("div");
-  equals(context.get(1), '<div>');
-  equals(context.get(2), '<'+'/div>');
+  equals(SC.RenderContext.escapeHTML(context.get(1)), SC.RenderContext.escapeHTML('<div>'));
+  equals(SC.RenderContext.escapeHTML(context.get(2)), SC.RenderContext.escapeHTML('<'+'/div>'));
 });
 
 test("should NOT emit self closing tag if no tag is passed", function() {
   context.tag();
-  equals(context.get(1), '<div>');
-  equals(context.get(2), '<'+'/div>');
+  equals(SC.RenderContext.escapeHTML(context.get(1)), SC.RenderContext.escapeHTML('<div>'));
+  equals(SC.RenderContext.escapeHTML(context.get(2)), SC.RenderContext.escapeHTML('<'+'/div>'));
 });

data/frameworks/sproutcore/frameworks/foundation/tests/system/render_context/update.js

@@ -137,20 +137,20 @@ module("SC.RenderContext#update - className", {
 test("does not change class names if retrieved but not edited", function() {
   context.classNames();
   context.update();
-  equals(elem.getAttribute("class"), "foo bar", "class");
+  equals(SC.$(elem).attr("class"), "foo bar", "class");
 });
 
 test("replaces class name if classNames edited", function() {
   context.classNames('bar baz'.w());
   context.update();
-  equals(elem.getAttribute("class"), "bar baz", "attribute");
+  equals(SC.$(elem).attr("class"), "bar baz", "attribute");
 });
 
 test("set class names override class attr", function() {
   context.attr("class", "bar");
   context.classNames('baz'.w());
   context.update();
-  equals(elem.getAttribute("class"), "baz", "should use classNames");
+  equals(SC.$(elem).attr("class"), "baz", "should use classNames");
 });
 
 // ..........................................................

data/frameworks/sproutcore/frameworks/foundation/tests/system/request.js

@@ -83,6 +83,33 @@ test("Test Asynchronous GET Request, auto-deserializing JSON", function() {
 
 });
 
+test("Test auto-deserializing malformed JSON", function() {
+  request = SC.Request.getUrl(sc_static('malformed.json')).set('isJSON', YES);
+
+  var timer = setTimeout(function() {
+    ok(false, 'response did not invoke notify()');
+    window.start();
+  }, 1000);
+  
+  request.notify(this, function(response) {
+    ok(SC.ok(response), 'response should not be error');
+    
+    try {
+      var body = response.get('body');
+      ok(!SC.ok(body), 'body should be an error');
+    } catch(e) {
+      ok(false, 'getting the body should not throw an exception');
+    }
+    
+    clearTimeout(timer);
+    window.start();
+  });
+  
+  request.send();
+
+  stop();
+});
+
 test("Test Synchronous GET Request, auto-deserializing JSON", function() {
   request.set("isAsynchronous", false);
   request.set("isJSON", true);

data/frameworks/sproutcore/frameworks/foundation/tests/system/utils/rect.js

@@ -0,0 +1,99 @@
+// ========================================================================
+// Rect utility Tests
+// ========================================================================
+
+
+module("Rect utilities");
+
+test("Get the X & Y points of a rect", function() {
+  var frame = { x: 50, y: 40, width: 700, height: 9000 };
+  expect(6);
+  equals(SC.minX(frame),50,'Left edge');
+  equals(SC.maxX(frame),750,'Right edge');
+  equals(SC.midX(frame),400,'Horizontal midpoint');
+  
+  equals(SC.minY(frame),40, 'Top edge');
+  equals(SC.maxY(frame),9040,'Bottom edge');
+  equals(SC.midY(frame),4540,'Vertical midpoint');
+});
+
+test("Treat empty object as frame with 0 width and height", function() {
+  var frame = { };
+  expect(6);
+  equals(SC.minX(frame),0,'Left edge');
+  equals(SC.maxX(frame),0,'Right edge');
+  equals(SC.midX(frame),0,'Horizontal midpoint');
+  
+  equals(SC.minY(frame),0,'Top edge');
+  equals(SC.maxY(frame),0,'Bottom edge');
+  equals(SC.midY(frame),0,'Vertical midpoint');
+});
+
+test("pointInRect() to test if a given point is inside the rect", function(){
+  var frame = { x: 50, y: 40, width: 700, height: 9000 };
+  
+  ok(SC.pointInRect({ x: 100, y: 100 }, frame), "Point in rect");
+  equals(NO, SC.pointInRect({ x: 40, y: 100 }, frame), "Point out of rect horizontally");
+  equals(NO, SC.pointInRect({ x: 600, y: 9100 }, frame), "Point out of rect vertically");
+  equals(NO, SC.pointInRect({ x: 0, y: 0 }, frame), "Point up and left from rect");
+  equals(NO, SC.pointInRect({ x: 800, y: 9500 }, frame), "Point down and right from rect");
+});
+
+test("rectsEqual() tests equality with default delta", function() {
+  var frame = { x: 50, y: 50, width: 100, height: 100 };
+  
+  equals(SC.rectsEqual(frame, frame), YES, "Frames are same object");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 100, height: 100 }), YES, "Frames have same position and dimensions");
+  equals(SC.rectsEqual(frame, { x: 50.08, y: 50, width: 100, height: 100 }), YES, "Frame.x above, within delta");
+  equals(SC.rectsEqual(frame, { x: 49.92, y: 50, width: 100, height: 100 }), YES, "Frame.x below, within delta");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50.099, width: 100, height: 100 }), YES, "Frame.y within delta");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 100.001, height: 100 }), YES, "Frame.width within delta");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 100, height: 100.09999 }), YES, "Frame.height within delta");
+  equals(SC.rectsEqual(frame, { x: 55, y: 50, width: 100, height: 100 }), NO, "Frame.x not equal");
+  equals(SC.rectsEqual(frame, { x: 50, y: 55, width: 100, height: 100 }), NO, "Frame.y not equal");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 105, height: 100 }), NO, "Frame.width not equal");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 100, height: 105 }), NO, "Frame.height not equal");
+});
+
+test("rectsEqual() tests equality with null delta", function() {
+  var frame = { x: 50, y: 50, width: 100, height: 100 };
+  
+  equals(SC.rectsEqual(frame, frame), YES, "Frames are same object");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 100, height: 100 }, null), YES, "Frames have same position and dimensions");
+  equals(SC.rectsEqual(frame, { x: 50.08, y: 50, width: 100, height: 100 }, null), YES, "Frame.x above, within delta");
+  equals(SC.rectsEqual(frame, { x: 49.92, y: 50, width: 100, height: 100 }, null), YES, "Frame.x below, within delta");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50.099, width: 100, height: 100 }, null), YES, "Frame.y within delta");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 100.001, height: 100 }, null), YES, "Frame.width within delta");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 100, height: 100.01 }, null), YES, "Frame.height within delta");
+  equals(SC.rectsEqual(frame, { x: 55, y: 50, width: 100, height: 100 }, null), NO, "Frame.x not equal");
+  equals(SC.rectsEqual(frame, { x: 50, y: 55, width: 100, height: 100 }, null), NO, "Frame.y not equal");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 105, height: 100 }, null), NO, "Frame.width not equal");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 100, height: 105 }, null), NO, "Frame.height not equal");
+});
+
+test("rectsEqual() tests equality with delta of 10", function() {
+  var frame = { x: 50, y: 50, width: 100, height: 100 };
+  
+  equals(SC.rectsEqual(frame, frame), YES, "Frames are same object");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 100, height: 100 }, 10), YES, "Frames have same position and dimensions");
+  equals(SC.rectsEqual(frame, { x: 59.99, y: 50, width: 100, height: 100 }, 10), YES, "Frame.x above, within delta");
+  equals(SC.rectsEqual(frame, { x: 41, y: 50, width: 100, height: 100 }, 10), YES, "Frame.x below, within delta");
+  equals(SC.rectsEqual(frame, { x: 50, y: 59, width: 100, height: 100 }, 10), YES, "Frame.y within delta");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 109, height: 100 }, 10), YES, "Frame.width within delta");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 100, height: 100.000002 }, 10), YES, "Frame.height within delta");
+  equals(SC.rectsEqual(frame, { x: 61, y: 50, width: 100, height: 100 }, 10), NO, "Frame.x not equal");
+  equals(SC.rectsEqual(frame, { x: 50, y: 92, width: 100, height: 100 }, 10), NO, "Frame.y not equal");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 89, height: 100 }, 10), NO, "Frame.width not equal");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 100, height: 89.99999 }, 10), NO, "Frame.height not equal");
+});
+
+test("rectsEqual() tests equality with delta of 0", function() {
+  var frame = { x: 50, y: 50, width: 100, height: 100 };
+  
+  equals(SC.rectsEqual(frame, frame), YES, "Frames are same object");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 100, height: 100 }, 0), YES, "Frames have same position and dimensions");
+  equals(SC.rectsEqual(frame, { x: 50.0001, y: 50, width: 100, height: 100 }, 0), NO, "Frame.x not equal");
+  equals(SC.rectsEqual(frame, { x: 50, y: 51, width: 100, height: 100 }, 0), NO, "Frame.y not equal");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 99, height: 100 }, 0), NO, "Frame.width not equal");
+  equals(SC.rectsEqual(frame, { x: 50, y: 50, width: 100, height: 102 }, 0), NO, "Frame.height not equal");
+});