Cleanup: Replace 'DurationInt' with int32, and always use 32-bit integers for tracking time.

base/profiler/tracked_time.h

Index: base/profiler/tracked_time.h
diff --git a/base/profiler/tracked_time.h b/base/profiler/tracked_time.h
index 5493e2068739b31ebc59c3ccf1e63ebab6364015..b96bfa25202315d7b297b615d622f66a0fafaaea 100644
--- a/base/profiler/tracked_time.h
+++ b/base/profiler/tracked_time.h
@@ -12,14 +12,8 @@
 
 namespace tracked_objects {
 
-typedef int DurationInt;
-
 //------------------------------------------------------------------------------
 
-#define USE_FAST_TIME_CLASS_FOR_DURATION_CALCULATIONS

[Ilya Sherman, 2012/03/21 21:47:49]

I should mention: I'm not sure if this #define is still something that you think
is valuable to toggle on and off.  If it is, I'll remove its removal from this
CL.

[jar (doing other things), 2012/03/23 05:28:58]

I don't see us going back... so this is fine to kill.  There was a tiny chance
that we'd make a faster time class (perhaps centered on the message loop...
using TLS there)... but if/when we do... this code will all be gone.

On 2012/03/21 21:47:49, Ilya Sherman wrote:

-
-#if defined(USE_FAST_TIME_CLASS_FOR_DURATION_CALCULATIONS)
-
 // TimeTicks maintains a wasteful 64 bits of data (we need less than 32), and on
 // windows, a 64 bit timer is expensive to even obtain. We use a simple
 // millisecond counter for most of our time values, as well as millisecond units
@@ -72,17 +66,6 @@ class BASE_EXPORT TrackedTime {  // Similar to base::TimeTicks.
   uint32 ms_;
 };
 
-#else
-
-// Just use full 64 bit time calculations, and the slower TimeTicks::Now().
-// This allows us (as an alternative) to test with larger ranges of times, and
-// with a more thoroughly tested class.
-
-typedef base::TimeTicks TrackedTime;
-typedef base::TimeDelta Duration;
-
-#endif  // USE_FAST_TIME_CLASS_FOR_DURATION_CALCULATIONS
-
 }  // namespace tracked_objects
 
 #endif  // BASE_PROFILER_TRACKED_TIME_H_