Bug 807829 - SkipRoot no longer needed for StackIter script
authorSteve Fink <sfink@mozilla.com>
Thu, 01 Nov 2012 13:57:47 -0700
changeset 112348 a0ecdce3d8ed94039d2cd1b81d5c2852de310722
parent 112347 8a88758ffc50dbb392c42a2e95d90e59268914ae
child 112349 20ae9d34616397ef4b9ba385818e9b77c916ff22
push id23812
push useremorley@mozilla.com
push dateTue, 06 Nov 2012 14:01:34 +0000
treeherdermozilla-central@f4aeed115e54 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs807829
milestone19.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 807829 - SkipRoot no longer needed for StackIter script
js/src/jsexn.cpp
--- a/js/src/jsexn.cpp
+++ b/js/src/jsexn.cpp
@@ -565,19 +565,16 @@ Exception(JSContext *cx, unsigned argc, 
         args[0].setString(message);
     } else {
         message = NULL;
     }
 
     /* Find the scripted caller. */
     NonBuiltinScriptFrameIter iter(cx);
 
-    /* XXX StackIter should not point directly to scripts. */
-    SkipRoot skip(cx, &iter);
-
     /* Set the 'fileName' property. */
     RootedScript script(cx, iter.script());
     RootedString filename(cx);
     if (args.length() > 1) {
         filename = ToString(cx, args[1]);
         if (!filename)
             return false;
         args[1].setString(filename);