Bug 866167 - GC: Make HashableValue::AutoRooter work r=terrence
authorJon Coppeard <jcoppeard@mozilla.com>
Fri, 26 Apr 2013 18:50:18 +0100
changeset 130104 acb4851041fbf289c904f937d60b9693d0413fa8
parent 130103 0d5eff7d8bb3f109c589542fe8c238f206a22641
child 130105 089bfd3486f5894317826476b4549fb2db04c0e9
push id24596
push userryanvm@gmail.com
push dateSat, 27 Apr 2013 01:20:57 +0000
treeherdermozilla-central@0e45f1b9521f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersterrence
bugs866167
milestone23.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 866167 - GC: Make HashableValue::AutoRooter work r=terrence
js/src/gc/RootMarking.cpp
--- a/js/src/gc/RootMarking.cpp
+++ b/js/src/gc/RootMarking.cpp
@@ -514,20 +514,18 @@ AutoGCRooter::trace(JSTracer *trc)
             mozilla::DebugOnly<RawObject> obj = e.front();
             MarkObjectRoot(trc, (RawObject *) &e.front(), "AutoObjectHashSet value");
             JS_ASSERT(obj == e.front());  // Needs rewriting for moving GC, see bug 726687.
         }
         return;
       }
 
       case HASHABLEVALUE: {
-          /*
         HashableValue::AutoRooter *rooter = static_cast<HashableValue::AutoRooter *>(this);
         rooter->trace(trc);
-          */
         return;
       }
 
       case IONMASM: {
 #ifdef JS_ION
         static_cast<js::ion::MacroAssembler::AutoRooter *>(this)->masm()->trace(trc);
 #endif
         return;