Support profiling of tasks run as sequenced_tasks

base/tracking_info.cc

 // Copyright (c) 2011 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 "base/tracking_info.h"
 
 #include "base/tracked_objects.h"
 
 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)
     : birth_tally(
           tracked_objects::ThreadData::TallyABirthIfActive(posted_from)),
       time_posted(TimeTicks::Now()),
       delayed_run_time(delayed_run_time) {
 }
 
 TrackingInfo::~TrackingInfo() {}
 
 }  // namespace base