Fix Simple Cache race on multiple dooms in flight.

net/disk_cache/simple/simple_entry_impl.cc

Index: net/disk_cache/simple/simple_entry_impl.cc
diff --git a/net/disk_cache/simple/simple_entry_impl.cc b/net/disk_cache/simple/simple_entry_impl.cc
index 3a54a1fa4c0e79d2dd1f7173b5f4aef272fbc4a5..fbda6cbe285618a2d8cac525dba784c7bee56aac 100644
--- a/net/disk_cache/simple/simple_entry_impl.cc
+++ b/net/disk_cache/simple/simple_entry_impl.cc
@@ -268,6 +268,8 @@ int SimpleEntryImpl::DoomEntry(const CompletionCallback& callback) {
   net_log_.AddEvent(net::NetLog::TYPE_SIMPLE_CACHE_ENTRY_DOOM_BEGIN);
 
   MarkAsDoomed();
+  if (backend_)

[pasko, 2013/09/16 14:01:42]

nit: we are inconsistent at how we check the weak pointer of the backend in this
file. I'd prefer to have backend_.get() instead sinse it better suggests that
the backend_.get() is a pointer and can go away later.

[gavinp, 2013/09/16 14:14:41]

Done.

+    backend_->OnDoomStart(entry_hash_);
   pending_operations_.push(SimpleEntryOperation::DoomOperation(this, callback));
   RunNextOperationIfNeeded();
   return net::ERR_IO_PENDING;
@@ -917,8 +919,6 @@ void SimpleEntryImpl::WriteDataInternal(int stream_index,
 }
 
 void SimpleEntryImpl::DoomEntryInternal(const CompletionCallback& callback) {
-  if (backend_)
-    backend_->OnDoomStart(entry_hash_);
   PostTaskAndReplyWithResult(
       worker_pool_, FROM_HERE,
       base::Bind(&SimpleSynchronousEntry::DoomEntry, path_, key_, entry_hash_),