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, 5 insertions, 2 deletions

diff --git a/chrome/browser/download_file.h b/chrome/browser/download_file.h
index b995281..f9c6e91 100644
--- a/chrome/browser/download_file.h
+++ b/chrome/browser/download_file.h
@@ -57,9 +57,12 @@ class GURL;
 class MessageLoop;
 class ResourceDispatcherHost;
 class Task;
-class Timer;
 class URLRequestContext;
 
+namespace base {
+class Timer;
+}
+
 // DownloadBuffer --------------------------------------------------------------
 
 // This container is created and populated on the io_thread, and passed to the
@@ -240,7 +243,7 @@ class DownloadFileManager
 
   // Throttle updates to the UI thread.
   Task* update_task_;
-  Timer* update_timer_;
+  base::Timer* update_timer_;
 
   // The MessageLoop that the DownloadManagers live on.
   MessageLoop* ui_loop_;