Destroying the temporary context used in evalcx and also doing a GC invalidates shape numbers, which kinda makes testing hard because it breaks shapeOf(evalcx("lazy")) === shapeOf(evalcx("lazy")); don't GC. rs=brendan
authorJeff Walden <jwalden@mit.edu>
Thu, 08 Jan 2009 17:54:04 -0800
changeset 23701 0e626f5d476f2c213b8759bbdf1bbabbfd7c0720
parent 23700 08915fd1c4427294e96c9c2a6e13930b854b842e
child 23702 080ccddea3b51d9d94a40e7a00e7cb42847495bd
push id4690
push userrsayre@mozilla.com
push dateThu, 15 Jan 2009 07:42:55 +0000
treeherdermozilla-central@ddfa483fea2a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbrendan
milestone1.9.2a1pre
Destroying the temporary context used in evalcx and also doing a GC invalidates shape numbers, which kinda makes testing hard because it breaks shapeOf(evalcx("lazy")) === shapeOf(evalcx("lazy")); don't GC. rs=brendan
js/src/js.cpp
--- a/js/src/js.cpp
+++ b/js/src/js.cpp
@@ -2673,17 +2673,17 @@ EvalInContext(JSContext *cx, JSObject *o
                 JS_ReportOutOfMemory(cx);
         }
     }
 
 out:
 #ifdef JS_THREADSAFE
     JS_EndRequest(scx);
 #endif
-    JS_DestroyContext(scx);
+    JS_DestroyContextNoGC(scx);
     return ok;
 }
 
 static int32 JS_FASTCALL
 ShapeOf_tn(JSObject *obj)
 {
     if (!obj)
         return 0;