Bug 1346725 - removed useless null check in DoSampleStackTrace. r?mstange
MozReview-Commit-ID: 374xPqkTMxJ
--- a/tools/profiler/core/platform.cpp
+++ b/tools/profiler/core/platform.cpp
@@ -931,17 +931,17 @@ DoNativeBacktrace(PS::LockRef aLock, Pro
static void
DoSampleStackTrace(PS::LockRef aLock, ProfileBuffer* aBuffer,
TickSample* aSample)
{
NativeStack nativeStack = { nullptr, nullptr, 0, 0 };
MergeStacksIntoProfile(aBuffer, aSample, nativeStack);
- if (aSample && gPS->FeatureLeaf(aLock)) {
+ if (gPS->FeatureLeaf(aLock)) {
aBuffer->addTag(ProfileBufferEntry::NativeLeafAddr((void*)aSample->pc));
}
}
// This function is called for each sampling period with the current program
// counter. It is called within a signal and so must be re-entrant.
static void
Tick(PS::LockRef aLock, ProfileBuffer* aBuffer, TickSample* aSample)