Bug 650161 - Unify the finalization and moving GC callbacks into a weak pointer update callback r=terrence r=bholley
This commit is contained in:
@@ -36,7 +36,6 @@ IdToObjectMap::trace(JSTracer *trc)
|
||||
for (Table::Range r(table_.all()); !r.empty(); r.popFront()) {
|
||||
DebugOnly<JSObject *> prior = r.front().value().get();
|
||||
JS_CallObjectTracer(trc, &r.front().value(), "ipc-object");
|
||||
MOZ_ASSERT(r.front().value() == prior);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -44,8 +43,9 @@ void
|
||||
IdToObjectMap::sweep()
|
||||
{
|
||||
for (Table::Enum e(table_); !e.empty(); e.popFront()) {
|
||||
DebugOnly<JSObject *> prior = e.front().value().get();
|
||||
if (JS_IsAboutToBeFinalized(&e.front().value()))
|
||||
JS::Heap<JSObject *> *objp = &e.front().value();
|
||||
JS_UpdateWeakPointerAfterGC(objp);
|
||||
if (!*objp)
|
||||
e.removeFront();
|
||||
}
|
||||
}
|
||||
@@ -94,12 +94,24 @@ ObjectToIdMap::init()
|
||||
return table_ && table_->init(32);
|
||||
}
|
||||
|
||||
void
|
||||
ObjectToIdMap::trace(JSTracer *trc)
|
||||
{
|
||||
for (Table::Enum e(*table_); !e.empty(); e.popFront()) {
|
||||
JSObject *obj = e.front().key();
|
||||
JS_CallUnbarrieredObjectTracer(trc, &obj, "ipc-object");
|
||||
if (obj != e.front().key())
|
||||
e.rekeyFront(obj);
|
||||
}
|
||||
}
|
||||
|
||||
void
|
||||
ObjectToIdMap::sweep()
|
||||
{
|
||||
for (Table::Enum e(*table_); !e.empty(); e.popFront()) {
|
||||
JSObject *obj = e.front().key();
|
||||
if (JS_IsAboutToBeFinalizedUnbarriered(&obj))
|
||||
JS_UpdateWeakPointerAfterGCUnbarriered(&obj);
|
||||
if (!obj)
|
||||
e.removeFront();
|
||||
else if (obj != e.front().key())
|
||||
e.rekeyFront(obj);
|
||||
@@ -568,9 +580,3 @@ JavaScriptShared::Wrap(JSContext *cx, HandleObject aObj, InfallibleTArray<CpowEn
|
||||
|
||||
return true;
|
||||
}
|
||||
void JavaScriptShared::fixupAfterMovingGC()
|
||||
{
|
||||
objects_.sweep();
|
||||
cpows_.sweep();
|
||||
objectIds_.sweep();
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user