Bug 833143 - Don't GC in nsXREDirProvider::DoShutdown. r=bsmedberg, r=billm

--HG--
extra : rebase_source : c3c6ca5f7d68ad773254a421c4a21c4472edd64f
extra : source : 5e01d4124eaefeb0702f31689a9223f14351368e

toolkit/xre/nsXREDirProvider.cpp

@@ -1098,15 +1098,6 @@ nsXREDirProvider::DoShutdown()
       static const char16_t kShutdownPersist[] = u"shutdown-persist";
       obsSvc->NotifyObservers(nullptr, "profile-change-net-teardown", kShutdownPersist);
       obsSvc->NotifyObservers(nullptr, "profile-change-teardown", kShutdownPersist);
-
-      // Phase 2c: Now that things are torn down, force JS GC so that things which depend on
-      // resources which are about to go away in "profile-before-change" are destroyed first.
-
-      if (JSContext* cx = dom::danger::GetJSContext()) {
-        JS_GC(cx);
-      }
-
-      // Phase 3: Notify observers of a profile change
       obsSvc->NotifyObservers(nullptr, "profile-before-change", kShutdownPersist);
       obsSvc->NotifyObservers(nullptr, "profile-before-change-qm", kShutdownPersist);
       obsSvc->NotifyObservers(nullptr, "profile-before-change-telemetry", kShutdownPersist);