Explicitly track the time source type in base/profiler/alternate_timer.h

base/profiler/alternate_timer.h

Index: base/profiler/alternate_timer.h
diff --git a/base/profiler/alternate_timer.h b/base/profiler/alternate_timer.h
index 778dc62e658722595492b505dfe6d2fce49ca0ca..5397e1957bd151df64f36e346dafe0ce412b474d 100644
--- a/base/profiler/alternate_timer.h
+++ b/base/profiler/alternate_timer.h
@@ -12,6 +12,11 @@
 
 namespace tracked_objects {
 
+BASE_EXPORT enum TimeSourceType {
+  TIME_SOURCE_TYPE_WALL_TIME,
+  TIME_SOURCE_TYPE_TCMALLOC
+};
+
 // Provide type for an alternate timer function.
 typedef unsigned int NowFunction();
 
@@ -24,11 +29,14 @@ extern const char kAlternateProfilerTime[];
 // profiling.  Typically this is called by an allocator that is providing a
 // function that indicates how much memory has been allocated on any given
 // thread.
-void SetAlternateTimeSource(NowFunction* now_function);
+void SetAlternateTimeSource(NowFunction* now_function, TimeSourceType type);
 
 // Gets the pointer to a function that was set via SetAlternateTimeSource().
 // Returns NULL if no set was done prior to calling GetAlternateTimeSource.
-BASE_EXPORT NowFunction* GetAlternateTimeSource();

[jar (doing other things), 2012/04/26 00:00:11]

I suspect you still need to say BASE_EXPORT here.

[Ilya Sherman, 2012/04/26 00:05:46]

I don't believe BASE_EXPORT is needed here anymore.  This used to be used by
metrics_log.cc, but now GetTimeSourceType() is instead.  The only other client
of this method is in base/, so there shouldn't be a need to export the method
outside the DLL.  We'll see if the try bots disagree :)

+NowFunction* GetAlternateTimeSource();
+
+// Returns the type of the currently set time source.
+BASE_EXPORT TimeSourceType GetTimeSourceType();
 
 }  // namespace tracked_objects