[tracing] Adding task information to heap profiler

components/timers/alarm_timer_chromeos.cc

 // Copyright 2014 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.
 
 #include "components/timers/alarm_timer_chromeos.h"
 
 #include <stdint.h>
 #include <sys/timerfd.h>
 #include <utility>
 
@@ skipping 416 matching lines @@
   // Take ownership of the pending user task, which is going to be cleared by
   // the Stop() or Reset() functions below.
   scoped_ptr<base::PendingTask> pending_user_task(std::move(pending_task_));
 
   // Re-schedule or stop the timer as requested.
   if (base::Timer::is_repeating())
     Reset();
   else
     Stop();
 
-  TRACE_TASK_EXECUTION("AlarmTimer::OnTimerFired", *pending_user_task);
+  TRACE_TASK_EXECUTION("AlarmTimer::OnTimerFired", (*pending_user_task));

[Primiano Tucci (use gerrit), 2016/03/31 15:37:16]

why do we need parentheses here? smells like we need paren in the macro?

[Nico, 2016/03/31 16:12:48]

yup, +1

[ssid, 2016/04/01 04:34:54]

yes, I was thinking it is better to add here instead of macro. moved it now.

 
   // Now run the user task.
   base::MessageLoop::current()->task_annotator()->RunTask("AlarmTimer::Reset",
                                                           *pending_user_task);
 }
 
 OneShotAlarmTimer::OneShotAlarmTimer() : AlarmTimer(false, false) {
 }
 
 OneShotAlarmTimer::~OneShotAlarmTimer() {
@@ skipping 18 matching lines @@
 SimpleAlarmTimer::SimpleAlarmTimer(const tracked_objects::Location& posted_from,
                                    base::TimeDelta delay,
                                    const base::Closure& user_task)
     : AlarmTimer(posted_from, delay, user_task, false) {
 }
 
 SimpleAlarmTimer::~SimpleAlarmTimer() {
 }
 
 }  // namespace timers