Support profiling of tasks run as sequenced_tasks

base/tracking_info.cc

Index: base/tracking_info.cc
===================================================================
--- base/tracking_info.cc	(revision 148495)
+++ base/tracking_info.cc	(working copy)
@@ -8,6 +8,10 @@
 
 namespace base {
 
+TrackingInfo::TrackingInfo()
+    : birth_tally(static_cast<tracked_objects::Births*>(NULL)) {

[akalin, 2012/07/27 17:45:39]

any particular reason for the explicit cast?  just for documentation purposes?

[jar (doing other things), 2012/07/27 18:15:46]

I had vague recollections of some compiler complaints... but maybe it was only
in the context of things like CHECK_EQ and such.  It certainly provides a bit of
documentation as well.

If you'd like, I can yank it and see how it does.

On 2012/07/27 17:45:39, akalin wrote:

[akalin, 2012/07/27 20:18:18]

I think I'd prefer to yank it out (and I expect it to work).  Having the cast
seems to me like it would make the reader question why the cast is needed.

Also, may as well #include <cstddef> (or stddef.h) for NULL.

[jar (doing other things), 2012/07/27 21:52:29]

Done.

+}
+
 TrackingInfo::TrackingInfo(
     const tracked_objects::Location& posted_from,
     base::TimeTicks delayed_run_time)