Index: base/trace_event/heap_profiler_allocation_context_tracker.h |
diff --git a/base/trace_event/heap_profiler_allocation_context_tracker.h b/base/trace_event/heap_profiler_allocation_context_tracker.h |
index 632a4eface8740a9415337c41c6d9874fb107367..a310cefc77e2546ea0a1db002b67d5bf35f6a05b 100644 |
--- a/base/trace_event/heap_profiler_allocation_context_tracker.h |
+++ b/base/trace_event/heap_profiler_allocation_context_tracker.h |
@@ -9,10 +9,19 @@ |
#include "base/atomicops.h" |
#include "base/base_export.h" |
+#include "base/debug/debugging_flags.h" |
+#include "base/debug/stack_trace.h" |
#include "base/logging.h" |
#include "base/macros.h" |
#include "base/trace_event/heap_profiler_allocation_context.h" |
+#if HAVE_TRACE_STACK_FRAME_POINTERS && !defined(OS_NACL) && \ |
+ (BUILDFLAG(ENABLE_PROFILING) || !defined(NDEBUG)) |
+#define ENABLE_NATIVE_ALLOCATION_TRACES 1 |
+#else |
+#define ENABLE_NATIVE_ALLOCATION_TRACES 0 |
+#endif |
+ |
namespace base { |
namespace trace_event { |
@@ -23,24 +32,32 @@ namespace trace_event { |
// details. |
class BASE_EXPORT AllocationContextTracker { |
public: |
- // Globally enables capturing allocation context. |
- // TODO(ruuda): Should this be replaced by |EnableCapturing| in the future? |
- // Or at least have something that guards agains enable -> disable -> enable? |
- static void SetCaptureEnabled(bool enabled); |
- |
- // Returns whether capturing allocation context is enabled globally. |
- inline static bool capture_enabled() { |
+ enum class CaptureMode: int32_t { |
+ DISABLED, // Don't capture anything |
+ PSEUDO_STACK, // GetContextSnapshot() returns pseudo stack trace |
+#if ENABLE_NATIVE_ALLOCATION_TRACES |
+ NATIVE_STACK // GetContextSnapshot() returns native (real) stack trace |
+#endif |
+ }; |
+ |
+ // Globally sets capturing mode. |
+ // TODO(primiano): How to guard against *_STACK -> DISABLED -> *_STACK? |
+ static void SetCaptureMode(CaptureMode mode); |
+ |
+ // Returns global capturing mode. |
+ inline static CaptureMode capture_mode() { |
// A little lag after heap profiling is enabled or disabled is fine, it is |
// more important that the check is as cheap as possible when capturing is |
// not enabled, so do not issue a memory barrier in the fast path. |
- if (subtle::NoBarrier_Load(&capture_enabled_) == 0) |
- return false; |
+ if (subtle::NoBarrier_Load(&capture_mode_) == |
+ static_cast<int32_t>(CaptureMode::DISABLED)) |
+ return CaptureMode::DISABLED; |
// In the slow path, an acquire load is required to pair with the release |
- // store in |SetCaptureEnabled|. This is to ensure that the TLS slot for |
+ // store in |SetCaptureMode|. This is to ensure that the TLS slot for |
// the thread-local allocation context tracker has been initialized if |
- // |capture_enabled| returns true. |
- return subtle::Acquire_Load(&capture_enabled_) != 0; |
+ // |capture_mode| returns something other than DISABLED. |
+ return static_cast<CaptureMode>(subtle::Acquire_Load(&capture_mode_)); |
} |
// Returns the thread-local instance, creating one if necessary. Returns |
@@ -80,7 +97,7 @@ class BASE_EXPORT AllocationContextTracker { |
private: |
AllocationContextTracker(); |
- static subtle::Atomic32 capture_enabled_; |
+ static subtle::Atomic32 capture_mode_; |
// The pseudo stack where frames are |TRACE_EVENT| names. |
std::vector<const char*> pseudo_stack_; |