Bug 1277052 - Remove the unused xpcomThreadsShutdown. r=Yoric
authorAndrew McCreight <continuation@gmail.com>
Wed, 01 Jun 2016 15:05:54 -0700
changeset 341043 8908cc624380f5a1790dc89ae5079c6f09611603
parent 341042 efcca7d36ad40386c137366b8f3161e13695e67f
child 341044 6115c96a870103809d586830da707155b0679df8
push id1183
push userraliiev@mozilla.com
push dateMon, 05 Sep 2016 20:01:49 +0000
treeherdermozilla-release@3148731bed45 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersYoric
bugs1277052
milestone49.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1277052 - Remove the unused xpcomThreadsShutdown. r=Yoric This should be xpcom-shutdown-threads, not xpcom-threads-shutdown, but it is unused so I just removed it.
toolkit/components/asyncshutdown/AsyncShutdown.jsm
toolkit/components/asyncshutdown/nsAsyncShutdown.js
toolkit/components/asyncshutdown/nsIAsyncShutdown.idl
--- a/toolkit/components/asyncshutdown/AsyncShutdown.jsm
+++ b/toolkit/components/asyncshutdown/AsyncShutdown.jsm
@@ -1031,13 +1031,12 @@ if (!isContent) {
 }
 
 // Don't add a barrier for content-child-shutdown because this
 // makes it easier to cause shutdown hangs.
 
 // All processes
 this.AsyncShutdown.webWorkersShutdown = getPhase("web-workers-shutdown");
 this.AsyncShutdown.xpcomWillShutdown = getPhase("xpcom-will-shutdown");
-this.AsyncShutdown.xpcomThreadsShutdown = getPhase("xpcom-threads-shutdown");
 
 this.AsyncShutdown.Barrier = Barrier;
 
 Object.freeze(this.AsyncShutdown);
--- a/toolkit/components/asyncshutdown/nsAsyncShutdown.js
+++ b/toolkit/components/asyncshutdown/nsAsyncShutdown.js
@@ -233,17 +233,16 @@ function nsAsyncShutdownService() {
     "sendTelemetry",
 
     // Child processes
     "contentChildShutdown",
 
     // All processes
     "webWorkersShutdown",
     "xpcomWillShutdown",
-    "xpcomThreadsShutdown",
     ]) {
     let k = _k;
     Object.defineProperty(this, k, {
       configurable: true,
       get: function() {
         delete this[k];
         let wrapped = AsyncShutdown[k]; // May be undefined, if we're on the wrong process.
         let result = wrapped ? new nsAsyncShutdownClient(wrapped) : undefined;
--- a/toolkit/components/asyncshutdown/nsIAsyncShutdown.idl
+++ b/toolkit/components/asyncshutdown/nsIAsyncShutdown.idl
@@ -208,18 +208,13 @@ interface nsIAsyncShutdownService: nsISu
   /**
    * Barrier for notification xpcom-will-shutdown.
    */
   readonly attribute nsIAsyncShutdownClient xpcomWillShutdown;
 
   // Don't add a barrier for content-child-shutdown because this
   // makes it easier to cause shutdown hangs.
 
-  /**
-   * Barrier for notification xpcom-threads-shutdown.
-   */
-  readonly attribute nsIAsyncShutdownClient xpcomThreadsShutdown;
-
 };
 
 %{C++
 #define NS_ASYNCSHUTDOWNSERVICE_CONTRACTID "@mozilla.org/async-shutdown-service;1"
 %}