Creating a framework for suppressing pollution of the profiler data

base/message_loop/message_loop.cc

Index: base/message_loop/message_loop.cc
diff --git a/base/message_loop/message_loop.cc b/base/message_loop/message_loop.cc
index dd1a393ab08633514df9d6d021a1be7ff4dbd23f..3e4a4b19645694ce808f016c32812f8cf4205476 100644
--- a/base/message_loop/message_loop.cc
+++ b/base/message_loop/message_loop.cc
@@ -416,7 +416,9 @@ bool MessageLoop::ProcessNextDelayedNonNestableTask() {
 
 void MessageLoop::RunTask(const PendingTask& pending_task) {
   tracked_objects::TrackedTime start_time =
-      tracked_objects::ThreadData::NowForStartOfRun(pending_task.birth_tally);
+     tracked_objects::ThreadData::NowForStartOfRun(pending_task.birth_tally);
+  tracked_objects::TaskStopwatch stopwatch;
+  stopwatch.Start(start_time);
 
   TRACE_EVENT_FLOW_END1(TRACE_DISABLED_BY_DEFAULT("toplevel.flow"),
       "MessageLoop::PostTask", TRACE_ID_MANGLE(GetTaskTraceID(pending_task)),
@@ -451,8 +453,9 @@ void MessageLoop::RunTask(const PendingTask& pending_task) {
   FOR_EACH_OBSERVER(TaskObserver, task_observers_,
                     DidProcessTask(pending_task));
 
-  tracked_objects::ThreadData::TallyRunOnNamedThreadIfTracking(pending_task,
-      start_time, tracked_objects::ThreadData::NowForEndOfRun());

[jar (doing other things), 2014/08/08 01:34:48]

It is pretty critical that NowForEndOFRun() is used, as there is a level of
indirection used here.  I'm wary that you may have dropped that level of
indirection.  

...but perhaps I've forgotten what this function did.  It used to either use
time, or allocation counts (as I recall).

Perhaps your stop watch also uses these methods.

[vadimt, 2014/08/08 20:39:33]

No, it was just a stupid mistake. I wanted to use this, but it slipped through
fingers.

+  stopwatch.Stop();
+  tracked_objects::ThreadData::TallyRunOnNamedThreadIfTracking(
+      pending_task, stopwatch);
 
   nestable_tasks_allowed_ = true;
 }