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..7f478acad5741764eb3c24a16f5880cdc56e56e6 100644
--- a/net/disk_cache/simple/simple_entry_impl.cc
+++ b/net/disk_cache/simple/simple_entry_impl.cc
@@ -565,6 +565,7 @@ void SimpleEntryImpl::MarkAsDoomed() {
     return;
   backend_->index()->Remove(entry_hash_);
   RemoveSelfFromBackend();
+  backend_->OnDoomStart(entry_hash_);

[gavinp, 2013/09/16 12:40:12]

This is a bug. We should only do this on actual doom operations.

 }
 
 void SimpleEntryImpl::RunNextOperationIfNeeded() {
@@ -917,8 +918,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_),