Move TransportSecurityPersister completely to IO thread.

chrome/browser/transport_security_persister.cc

Index: chrome/browser/transport_security_persister.cc
diff --git a/chrome/browser/transport_security_persister.cc b/chrome/browser/transport_security_persister.cc
index 59f093a31451cf0f10885359808bde497721f94e..d1fd6e142aa8d5f18b37be2197bdf34fd35e2b59 100644
--- a/chrome/browser/transport_security_persister.cc
+++ b/chrome/browser/transport_security_persister.cc
@@ -20,12 +20,14 @@ TransportSecurityPersister::TransportSecurityPersister(
       writer_(profile_path.AppendASCII("TransportSecurity"),
               BrowserThread::GetMessageLoopProxyForThread(BrowserThread::FILE)),
       readonly_(readonly) {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   transport_security_state_->SetDelegate(this);
 }
 
 TransportSecurityPersister::~TransportSecurityPersister() {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
+
   if (writer_.HasPendingWrite())
     writer_.DoScheduledWrite();
 
@@ -33,7 +35,7 @@ TransportSecurityPersister::~TransportSecurityPersister() {
 }
 
 void TransportSecurityPersister::Init() {
-  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   BrowserThread::PostTask(

[willchan no longer on Chromium, 2011/09/20 01:29:51]

Can you use BrowserThread::PostTaskAndReply() and pass a base::WeakPtr instead
to TransportSecurityPersister::Load()? I think that will help eliminate the need
for the refcounting here.

       BrowserThread::FILE, FROM_HERE,
       NewRunnableMethod(this, &TransportSecurityPersister::Load));
@@ -75,5 +77,6 @@ void TransportSecurityPersister::StateIsDirty(
 }
 
 bool TransportSecurityPersister::SerializeData(std::string* data) {
+  DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   return transport_security_state_->Serialise(data);
 }