Add TimeDelay interfaces to TestTimeouts.

Add TimeDelay interfaces to TestTimeouts.

R=brettw@chromium.org
BUG=108171
TEST=


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

[Ted Vessenes, 2012-01-02 22:34:41]

It seemed pretty clear to me that this class should define TimeDelta interfaces.
 But do you think the integer millisecond interfaces should get removed in the
future, or is there a compelling reason to keep both versions of the functions?

[brettw, 2012-01-03 05:38:48]

I don't think we should have two versions of the functions.

http://codereview.chromium.org/8952022/diff/1/base/test/test_timeouts.h
File base/test/test_timeouts.h (right):

http://codereview.chromium.org/8952022/diff/1/base/test/test_timeouts.h#newco...
base/test/test_timeouts.h:12: using base::TimeDelta;
You can't do this in a header file.

[Ted Vessenes, 2012-01-03 15:14:17]

Sorry about that; the 'using' statement is a bit new to me, but upon reflection
it makes perfect sense why that doesn't work.  I'll add these functions to the
list of things to remove at the end of this bug fix.

[brettw, 2012-01-03 17:40:00]

lgtm

[commit-bot: I haz the power, 2012-01-03 17:43:59]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tedvessenes@gmail.com/8952022/6001

[commit-bot: I haz the power, 2012-01-03 19:28:21]

Try job failure for 8952022-6001 (retry) on mac_rel for step "ui_tests" (clobber
build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-01-03 19:49:30]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tedvessenes@gmail.com/8952022/6001

[commit-bot: I haz the power, 2012-01-03 21:04:07]

Change committed as 116194