Fix up SequencedWorkerPool in preparation for making it a TaskRunner

webkit/dom_storage/dom_storage_task_runner.h

Index: webkit/dom_storage/dom_storage_task_runner.h
diff --git a/webkit/dom_storage/dom_storage_task_runner.h b/webkit/dom_storage/dom_storage_task_runner.h
index a5c09b1cf73d667b50936fa5edfb95b86cf1f122..6384d9eea3492a6ef56a8091b516798cefaabaca 100644
--- a/webkit/dom_storage/dom_storage_task_runner.h
+++ b/webkit/dom_storage/dom_storage_task_runner.h
@@ -8,6 +8,7 @@
 
 #include "base/memory/ref_counted.h"
 #include "base/threading/sequenced_worker_pool.h"
+#include "base/time.h"
 
 namespace base {
 class MessageLoopProxy;
@@ -21,7 +22,8 @@ namespace dom_storage {
 class DomStorageTaskRunner
     : public base::RefCountedThreadSafe<DomStorageTaskRunner> {
  public:
-  explicit DomStorageTaskRunner(base::MessageLoopProxy* message_loop);
+  explicit DomStorageTaskRunner(
+      const scoped_refptr<base::MessageLoopProxy>& message_loop);
   virtual ~DomStorageTaskRunner();
 
   // Schedules a task to be run immediately.
@@ -36,7 +38,7 @@ class DomStorageTaskRunner
       base::TimeDelta delay);
 
  protected:
-  scoped_refptr<base::MessageLoopProxy> message_loop_;
+  const scoped_refptr<base::MessageLoopProxy> message_loop_;
 };
 
 // A derived class that utlizes the SequenceWorkerPool under a
@@ -45,8 +47,8 @@ class DomStorageTaskRunner
 class DomStorageWorkerPoolTaskRunner : public DomStorageTaskRunner {
  public:
   DomStorageWorkerPoolTaskRunner(
-      base::SequencedWorkerPool* sequenced_worker_pool,
-      base::MessageLoopProxy* delayed_task_loop);
+      const scoped_refptr<base::SequencedWorkerPool>& sequenced_worker_pool,
+      const scoped_refptr<base::MessageLoopProxy>& delayed_task_loop);

[michaeln, 2012/02/23 03:55:25]

Has our style been changed to require this technique of passing refs to
refcounted objects? Unless this is specificially required, i'd prefer the simple
ptr types to remain here.

[akalin, 2012/02/23 22:22:30]

Done.

   virtual ~DomStorageWorkerPoolTaskRunner();
 
   // Schedules a sequenced worker task to be run immediately.
@@ -61,7 +63,7 @@ class DomStorageWorkerPoolTaskRunner : public DomStorageTaskRunner {
       base::TimeDelta delay) OVERRIDE;
 
  private:
-  base::SequencedWorkerPool* sequenced_worker_pool_;  // not owned
+  const scoped_refptr<base::SequencedWorkerPool> sequenced_worker_pool_;
   base::SequencedWorkerPool::SequenceToken sequence_token_;
 };