Bug 1342306 (part 2a) - Fix a comment that part 2 made out-of-date. r=me.
authorNicholas Nethercote <nnethercote@mozilla.com>
Mon, 06 Mar 2017 12:53:37 +1100
changeset 346044 d01cc63b4504
parent 346043 7c3ed29c6ffb
child 346045 86ba78dc87ce
push id31452
push usercbook@mozilla.com
push date2017-03-06 09:54 +0000
treeherdermozilla-central@966464a68a2c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs1342306
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 1342306 (part 2a) - Fix a comment that part 2 made out-of-date. r=me. DONTBUILD because it's a comment-only change.
tools/profiler/public/PseudoStack.h
--- a/tools/profiler/public/PseudoStack.h
+++ b/tools/profiler/public/PseudoStack.h
@@ -415,18 +415,17 @@ private:
   // been previously observed. This is used for an optimization: in some cases,
   // when a thread is asleep, we duplicate the previous sample, which is
   // cheaper than taking a new sample.
   //
   // mSleep is atomic because it is accessed from multiple threads.
   //
   // - It is written only by this thread, via setSleeping() and setAwake().
   //
-  // - It is read by the SamplerThread (on Win32 and Mac) or the SigprofSender
-  //   thread (on Linux and Android).
+  // - It is read by SamplerThread::Run().
   //
   // There are two cases where racing between threads can cause an issue.
   //
   // - If CanDuplicateLastSampleDueToSleep() returns false but that result is
   //   invalidated before being acted upon, we will take a full sample
   //   unnecessarily. This is additional work but won't cause any correctness
   //   issues. (In actual fact, this case is impossible. In order to go from
   //   CanDuplicateLastSampleDueToSleep() returning false to it returning true