Simplify OneShotTimer and RepeatingTimer. Fix up all consumers.

Major changes:

OneShotTimer and RepeatingTimer become template classes that no longer require
a Task or a Timer object.  They just use PostDelayedTask.  Under the hood that
still uses a Timer object.

The API is much simpler for consumers as they now no longer need to worry about
allocating a Task or managing the lifetime of the object pointer held by the
Task.

I added some new unit tests to timer_unittest.cc to cover the API.

I preserved the old TimerManager / Timer API for now, but I plan to soon kill
it.

R=brettw
BUG=1346553



git-svn-id: svn://svn.chromium.org/chrome/trunk/src@1502 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 1 deletions

diff --git a/chrome/views/throbber.h b/chrome/views/throbber.h
index d7e298c..812cae1 100644
--- a/chrome/views/throbber.h
+++ b/chrome/views/throbber.h
@@ -12,7 +12,10 @@
 #include "chrome/views/view.h"
 
 class SkBitmap;
+
+namespace base {
 class Timer;
+}
 
 namespace ChromeViews {
 
@@ -49,7 +52,7 @@ class Throbber : public ChromeViews::View,
   DWORD last_time_recorded_;
   SkBitmap* frames_;
   int frame_time_ms_;
-  Timer* timer_;
+  base::Timer* timer_;
 
   DISALLOW_EVIL_CONSTRUCTORS(Throbber);
 };