base::Bind fixes

net/base/default_origin_bound_cert_store.h

Index: net/base/default_origin_bound_cert_store.h
diff --git a/net/base/default_origin_bound_cert_store.h b/net/base/default_origin_bound_cert_store.h
index 38a4a56e4c30a98182105adb45c1ac69980fa099..05dd70bd3114a3c04c78689f0cd6cbd1fd6dc893 100644
--- a/net/base/default_origin_bound_cert_store.h
+++ b/net/base/default_origin_bound_cert_store.h
@@ -10,6 +10,7 @@
 #include <string>
 #include <vector>
 
+#include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/memory/ref_counted.h"
 #include "base/synchronization/lock.h"
@@ -51,7 +52,7 @@ class NET_EXPORT DefaultOriginBoundCertStore : public OriginBoundCertStore {
   // It may be posted to the current thread, or it may run on the thread that
   // actually does the flushing. Your Task should generally post a notification
   // to the thread you actually want to be notified on.
-  void FlushStore(Task* completion_task);
+  void FlushStore(const base::Closure& completion_task);

[wtc, 2011/11/16 06:30:18]

Should we rename the function parameter because it is no longer
a Task?

[groby-ooo-7-16, 2011/11/16 22:24:58]

We still call it a task pretty much everywhere else. (i.e. PostTask etc.)

 
   // OriginBoundCertStore implementation.
   virtual bool GetOriginBoundCert(const std::string& origin,
@@ -154,7 +155,7 @@ class NET_EXPORT DefaultOriginBoundCertStore::PersistentStore
   virtual void SetClearLocalStateOnExit(bool clear_local_state) = 0;
 
   // Flush the store and post the given Task when complete.
-  virtual void Flush(Task* completion_task) = 0;
+  virtual void Flush(const base::Closure& completion_task) = 0;
 
  protected:
   PersistentStore();