Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(133)

Side by Side Diff: base/trace_event/heap_profiler_allocation_context_tracker.h

Issue 1846333002: Revert of [tracing] Adding task information to heap profiler (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef BASE_TRACE_EVENT_HEAP_PROFILER_ALLOCATION_CONTEXT_TRACKER_H_ 5 #ifndef BASE_TRACE_EVENT_HEAP_PROFILER_ALLOCATION_CONTEXT_TRACKER_H_
6 #define BASE_TRACE_EVENT_HEAP_PROFILER_ALLOCATION_CONTEXT_TRACKER_H_ 6 #define BASE_TRACE_EVENT_HEAP_PROFILER_ALLOCATION_CONTEXT_TRACKER_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/atomicops.h" 10 #include "base/atomicops.h"
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 // Set the thread name in the AllocationContextTracker of current thread if 51 // Set the thread name in the AllocationContextTracker of current thread if
52 // capture is enabled. 52 // capture is enabled.
53 static void SetCurrentThreadName(const char* name); 53 static void SetCurrentThreadName(const char* name);
54 54
55 // Pushes a frame onto the thread-local pseudo stack. 55 // Pushes a frame onto the thread-local pseudo stack.
56 void PushPseudoStackFrame(StackFrame frame); 56 void PushPseudoStackFrame(StackFrame frame);
57 57
58 // Pops a frame from the thread-local pseudo stack. 58 // Pops a frame from the thread-local pseudo stack.
59 void PopPseudoStackFrame(StackFrame frame); 59 void PopPseudoStackFrame(StackFrame frame);
60 60
61 // Push and pop current task's context. A stack is used to support nested
62 // tasks and the top of the stack will be used in allocation context.
63 void PushCurrentTaskContext(const char* context);
64 void PopCurrentTaskContext(const char* context);
65
66 // Returns a snapshot of the current thread-local context. 61 // Returns a snapshot of the current thread-local context.
67 AllocationContext GetContextSnapshot(); 62 AllocationContext GetContextSnapshot();
68 63
69 ~AllocationContextTracker(); 64 ~AllocationContextTracker();
70 65
71 private: 66 private:
72 AllocationContextTracker(); 67 AllocationContextTracker();
73 68
74 static subtle::Atomic32 capture_enabled_; 69 static subtle::Atomic32 capture_enabled_;
75 70
76 // The pseudo stack where frames are |TRACE_EVENT| names. 71 // The pseudo stack where frames are |TRACE_EVENT| names.
77 std::vector<StackFrame> pseudo_stack_; 72 std::vector<StackFrame> pseudo_stack_;
78 73
79 // The thread name is used as the first entry in the pseudo stack. 74 // The thread name is used as the first entry in the pseudo stack.
80 const char* thread_name_; 75 const char* thread_name_;
81 76
82 // Stack of tasks' contexts. Context serves as a different dimension than
83 // pseudo stack to cluster allocations.
84 std::vector<const char*> task_contexts_;
85
86 DISALLOW_COPY_AND_ASSIGN(AllocationContextTracker); 77 DISALLOW_COPY_AND_ASSIGN(AllocationContextTracker);
87 }; 78 };
88 79
89 } // namespace trace_event 80 } // namespace trace_event
90 } // namespace base 81 } // namespace base
91 82
92 #endif // BASE_TRACE_EVENT_HEAP_PROFILER_ALLOCATION_CONTEXT_TRACKER_H_ 83 #endif // BASE_TRACE_EVENT_HEAP_PROFILER_ALLOCATION_CONTEXT_TRACKER_H_
OLDNEW
« no previous file with comments | « base/trace_event/common/trace_event_common.h ('k') | base/trace_event/heap_profiler_allocation_context_tracker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698