Bug 1323100 - Stop double-registering the LazyIdleThread with the profiler. r?froydnj draft
authorMarkus Stange <mstange@themasta.com>
Wed, 21 Dec 2016 23:07:26 +0100
changeset 456892 95bf7ae1196bfb51cbd72099be926d91d4436acf
parent 456891 b63f74fa3326ea29a6f91997c6ec46909758c803
child 456893 fcc61ab1ce2362e92bc3efe3bdf8a69842f2d621
push id40636
push userbmo:mstange@themasta.com
push dateFri, 06 Jan 2017 12:38:48 +0000
reviewersfroydnj
bugs1323100
milestone53.0a1
Bug 1323100 - Stop double-registering the LazyIdleThread with the profiler. r?froydnj MozReview-Commit-ID: 2vdcgCcdOYJ
xpcom/threads/LazyIdleThread.cpp
--- a/xpcom/threads/LazyIdleThread.cpp
+++ b/xpcom/threads/LazyIdleThread.cpp
@@ -175,21 +175,16 @@ LazyIdleThread::EnsureThread()
   }
 
   return NS_OK;
 }
 
 void
 LazyIdleThread::InitThread()
 {
-  char aLocal;
-  profiler_register_thread(mName.get(), &aLocal);
-
-  PR_SetCurrentThreadName(mName.get());
-
   // Happens on mThread but mThread may not be set yet...
 
   nsCOMPtr<nsIThreadInternal> thread(do_QueryInterface(NS_GetCurrentThread()));
   MOZ_ASSERT(thread, "This should always succeed!");
 
   if (NS_FAILED(thread->SetObserver(this))) {
     NS_WARNING("Failed to set thread observer!");
   }
@@ -206,18 +201,16 @@ LazyIdleThread::CleanupThread()
   }
 
   {
     MutexAutoLock lock(mMutex);
 
     MOZ_ASSERT(!mThreadIsShuttingDown, "Shouldn't be true ever!");
     mThreadIsShuttingDown = true;
   }
-
-  profiler_unregister_thread();
 }
 
 void
 LazyIdleThread::ScheduleTimer()
 {
   ASSERT_OWNING_THREAD();
 
   bool shouldSchedule;