Bug 531225 - 'Workers: Share strings across thread boundary'. Fix shutdown leak
authorBen Turner <bent.mozilla@gmail.com>
Tue, 23 Feb 2010 09:57:14 -0800
changeset 38427 59622926cd6bdf7389c842db3c113b5e439bc16f
parent 38426 e65d3f9b0ede055ef6baa7e08f3cd286e4218c2e
child 38428 1918f1187eb64dac7bd02d5a1bd1d7a35d34936e
push id11753
push userbturner@mozilla.com
push dateTue, 23 Feb 2010 17:57:28 +0000
treeherdermozilla-central@59622926cd6b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs531225
milestone1.9.3a2pre
Bug 531225 - 'Workers: Share strings across thread boundary'. Fix shutdown leak
dom/src/threads/nsDOMThreadService.cpp
--- a/dom/src/threads/nsDOMThreadService.cpp
+++ b/dom/src/threads/nsDOMThreadService.cpp
@@ -875,25 +875,16 @@ nsDOMThreadService::Cleanup()
       JS_GC(safeContext);
     }
     NS_RELEASE(gThreadJSContextStack);
   }
 
   // These must be released after the thread pool is shut down.
   NS_IF_RELEASE(gJSRuntimeService);
   NS_IF_RELEASE(gWorkerSecurityManager);
-
-  if (sStringFinalizerIndex != -1) {
-#ifdef DEBUG
-    int index =
-#endif
-    JS_RemoveExternalStringFinalizer(StringFinalizer);
-    NS_ASSERTION(index == sStringFinalizerIndex, "Bad index!");
-    sStringFinalizerIndex = -1;
-  }
 }
 
 nsresult
 nsDOMThreadService::Dispatch(nsDOMWorker* aWorker,
                              nsIRunnable* aRunnable,
                              PRIntervalTime aTimeoutInterval,
                              PRBool aClearQueue)
 {