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

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

We were previously implicitly tracking the time source as either default or tcmalloc-based; but as we add more time sources (e.g. cpu time), we'll want to have explicit tracking for this.

BUG=none
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=134041

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

See nit below.  I'm *guessing* this will break some funky dll builds if you
don't get this stuff dead right.

Other than that... LGTM.

http://codereview.chromium.org/10225017/diff/2001/base/profiler/alternate_tim...
File base/profiler/alternate_timer.h (left):

http://codereview.chromium.org/10225017/diff/2001/base/profiler/alternate_tim...
base/profiler/alternate_timer.h:31: BASE_EXPORT NowFunction*
GetAlternateTimeSource();
I suspect you still need to say BASE_EXPORT here.

[Ilya Sherman, 2012-04-26 00:05:45]

https://chromiumcodereview.appspot.com/10225017/diff/2001/base/profiler/alter...
File base/profiler/alternate_timer.h (left):

https://chromiumcodereview.appspot.com/10225017/diff/2001/base/profiler/alter...
base/profiler/alternate_timer.h:31: BASE_EXPORT NowFunction*
GetAlternateTimeSource();
On 2012/04/26 00:00:11, jar wrote:
> I suspect you still need to say BASE_EXPORT here.

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 :)

[commit-bot: I haz the power, 2012-04-26 00:06:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/isherman@chromium.org/10225017/2001

[commit-bot: I haz the power, 2012-04-26 01:40:55]

Change committed as 134041