Bug 1615145 - Cleanup use of cellIter in TraceLogger. r=jandem

The ZoneCellIter will already check for finalization so we don't need to
worry about checking again.

Depends on D62681

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

--HG--
extra : moz-landing-system : lando

js/src/jit/BaselineJIT.cpp

@@ -994,11 +994,8 @@ void jit::ToggleBaselineProfiling(JSContext* cx, bool enable) {
 #ifdef JS_TRACE_LOGGING
 void jit::ToggleBaselineTraceLoggerScripts(JSRuntime* runtime, bool enable) {
   for (ZonesIter zone(runtime, SkipAtoms); !zone.done(); zone.next()) {
-    for (auto iter = zone->cellIter<JSScript>(); !iter.done(); iter.next()) {
-      JSScript* script = iter;
-      if (gc::IsAboutToBeFinalizedUnbarriered(&script)) {
-        continue;
-      }
+    for (auto script = zone->cellIter<JSScript>(); !script.done();
+         script.next()) {
       if (!script->hasBaselineScript()) {
         continue;
       }
@@ -1009,11 +1006,8 @@ void jit::ToggleBaselineTraceLoggerScripts(JSRuntime* runtime, bool enable) {
 
 void jit::ToggleBaselineTraceLoggerEngine(JSRuntime* runtime, bool enable) {
   for (ZonesIter zone(runtime, SkipAtoms); !zone.done(); zone.next()) {
-    for (auto iter = zone->cellIter<JSScript>(); !iter.done(); iter.next()) {
-      JSScript* script = iter;
-      if (gc::IsAboutToBeFinalizedUnbarriered(&script)) {
-        continue;
-      }
+    for (auto script = zone->cellIter<JSScript>(); !script.done();
+         script.next()) {
       if (!script->hasBaselineScript()) {
         continue;
       }