Bug 1557928 - Fix obsolete comments r=jonco

IsMarkingTracer no-longer exists, so remove one comment that refers to it
and fix another comment.

Differential Revision: https://phabricator.services.mozilla.com/D34674

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Paul Bone 2019-06-13 01:11:56 +00:00
parent 256fa598e8
commit 314393ad7b
2 changed files with 1 additions and 5 deletions

View File

@ -232,10 +232,6 @@ void js::CheckTracedThing(JSTracer* trc, T* thing) {
MapTypeToTraceKind<typename mozilla::RemovePointer<T>::Type>::kind ==
thing->getTraceKind());
/*
* Do not check IsMarkingTracer directly -- it should only be used in paths
* where we cannot be the gray buffering tracer.
*/
bool isGcMarkingTracer = trc->isMarkingTracer();
MOZ_ASSERT_IF(zone->requireGCTracer(),

View File

@ -157,7 +157,7 @@ static inline bool IsMarkingTrace(JSTracer* trc) {
//
// 1. During TraceRuntime, RuntimeHeapIsBusy() is true, but the
// tracer might not be a marking tracer.
// 2. When a write barrier executes, IsMarkingTracer is true, but
// 2. When a write barrier executes, isMarkingTracer is true, but
// RuntimeHeapIsBusy() will be false.
return JS::RuntimeHeapIsCollecting() && trc->isMarkingTracer();