[tracing] Ignore tracing allocations in heap profiler

base/trace_event/heap_profiler_allocation_context_tracker.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef BASE_TRACE_EVENT_HEAP_PROFILER_ALLOCATION_CONTEXT_TRACKER_H_
 #define BASE_TRACE_EVENT_HEAP_PROFILER_ALLOCATION_CONTEXT_TRACKER_H_
 
 #include <vector>
 
 #include "base/atomicops.h"
@@ skipping 34 matching lines @@
 
   // Returns the thread-local instance, creating one if necessary. Returns
   // always a valid instance, unless it is called re-entrantly, in which case
   // returns nullptr in the nested calls.
   static AllocationContextTracker* GetInstanceForCurrentThread();
 
   // Set the thread name in the AllocationContextTracker of the current thread
   // if capture is enabled.
   static void SetCurrentThreadName(const char* name);
 
+  void start_ignore_scope() { ignore_scope_count_++; }

[Primiano Tucci (use gerrit), 2016/04/20 16:34:57]

add a comment plz. One for both is ok.

[ssid, 2016/04/21 01:07:46]

Sorry uploaded older patch here.

+  void end_ignore_scope() {

[Primiano Tucci (use gerrit), 2016/04/20 16:34:57]

s/start/begin/
s/start/end/

[ssid, 2016/04/21 01:07:46]

I think you meant begin and end here.

+    DCHECK(ignore_scope_count_);
+    ignore_scope_count_--;
+  }
+
   // Pushes a frame onto the thread-local pseudo stack.
   void PushPseudoStackFrame(StackFrame frame);
 
   // Pops a frame from the thread-local pseudo stack.
   void PopPseudoStackFrame(StackFrame frame);
 
   // Push and pop current task's context. A stack is used to support nested
   // tasks and the top of the stack will be used in allocation context.
   void PushCurrentTaskContext(const char* context);
   void PopCurrentTaskContext(const char* context);
@@ skipping 11 matching lines @@
   // The pseudo stack where frames are |TRACE_EVENT| names.
   std::vector<StackFrame> pseudo_stack_;
 
   // The thread name is used as the first entry in the pseudo stack.
   const char* thread_name_;
 
   // Stack of tasks' contexts. Context serves as a different dimension than
   // pseudo stack to cluster allocations.
   std::vector<const char*> task_contexts_;
 
+  uint32_t ignore_scope_count_;

[Primiano Tucci (use gerrit), 2016/04/20 16:34:57]

s/count/depth/

[Dmitry Skiba, 2016/04/20 16:59:15]

Also, size_t is better here.

[ssid, 2016/04/21 01:07:46]

Done.

[ssid, 2016/04/21 01:07:46]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(AllocationContextTracker);
 };
 
 }  // namespace trace_event
 }  // namespace base
 
 #endif  // BASE_TRACE_EVENT_HEAP_PROFILER_ALLOCATION_CONTEXT_TRACKER_H_