Bug 1339695 (part 6) - Remove some B2G-only code in profiler_register_thread(). r=mstange.
authorNicholas Nethercote <nnethercote@mozilla.com>
Wed, 15 Feb 2017 16:26:23 +1100
changeset 343218 eb3b5ef730656f41a31d5b537a83553337c4a549
parent 343217 6205142186156da59305508759cd08982e38ae1b
child 343219 6e70c857f61b33950b695f2dbb1f7b6351a3c397
push id31372
push usercbook@mozilla.com
push dateThu, 16 Feb 2017 12:16:10 +0000
treeherdermozilla-central@2737f66ad6ac [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmstange
bugs1339695
milestone54.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 1339695 (part 6) - Remove some B2G-only code in profiler_register_thread(). r=mstange.
tools/profiler/core/platform.cpp
--- a/tools/profiler/core/platform.cpp
+++ b/tools/profiler/core/platform.cpp
@@ -2343,25 +2343,16 @@ void
 profiler_register_thread(const char* aName, void* aGuessStackTop)
 {
   // This function runs both on and off the main thread.
 
   if (gInitCount == 0) {
     return;
   }
 
-#if defined(MOZ_WIDGET_GONK) && !defined(MOZ_PROFILING)
-  // The only way to profile secondary threads on b2g
-  // is to build with profiling OR have the profiler
-  // running on startup.
-  if (!profiler_is_active()) {
-    return;
-  }
-#endif
-
   MOZ_ASSERT(tlsPseudoStack.get() == nullptr);
   PseudoStack* stack = new PseudoStack();
   tlsPseudoStack.set(stack);
   bool isMainThread = is_main_thread_name(aName);
   void* stackTop = GetStackTop(aGuessStackTop);
   RegisterCurrentThread(aName, stack, isMainThread, stackTop);
 }