Fix bug 643839. r=brendan, a=clegnitto
authorBrian Hackett <bhackett1024@gmail.com>
Fri, 22 Apr 2011 12:29:30 -0700
changeset 68054 bfdb6e623a36158abce0cbae1d89713236acd0d6
parent 68053 768ec7eec906e61edcea29532ad8534ce63cbbf2
child 68055 ea8b7e3f457baed08cdf980b98705beabfd7ff56
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbrendan, clegnitto
bugs643839
milestone5.0a2
Fix bug 643839. r=brendan, a=clegnitto
js/src/jsscope.cpp
--- a/js/src/jsscope.cpp
+++ b/js/src/jsscope.cpp
@@ -961,20 +961,18 @@ JSObject::putProperty(JSContext *cx, jsi
      * Can't fail now, so free the previous incarnation's slot if the new shape
      * has no slot. But we do not need to free oldSlot (and must not, as trying
      * to will botch an assertion in JSObject::freeSlot) if the new lastProp
      * (shape here) has a slotSpan that does not cover it.
      */
     if (hadSlot && !shape->hasSlot()) {
         if (oldSlot < shape->slotSpan)
             freeSlot(cx, oldSlot);
-#ifdef DEBUG
         else
             getSlotRef(oldSlot).setUndefined();
-#endif
         JS_ATOMIC_INCREMENT(&cx->runtime->propertyRemovals);
     }
 
     CHECK_SHAPE_CONSISTENCY(this);
     METER(puts);
 
     const Shape *newShape = js_UpdateWatchpointsForShape(cx, this, shape);
     if (!newShape)