Don't doom the wrong simple cache entry.

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 e4e7220af7039c26924b58fb4737de40e995aaee..5bdf2d4de8277d7a4ceac10cf01790555c4608fc 100644
--- a/net/disk_cache/simple/simple_entry_impl.cc
+++ b/net/disk_cache/simple/simple_entry_impl.cc
@@ -253,6 +253,8 @@ int SimpleEntryImpl::CreateEntry(Entry** out_entry,
 }
 
 int SimpleEntryImpl::DoomEntry(const CompletionCallback& callback) {
+  if (doomed_)

[pasko, 2013/09/12 10:46:48]

hm, so if the MarkAsDoomed() on the previous entry was called while the backend
was gone, we would still now go and doom the wrong entry? Or we could not doom
the wrong one because the wrong one could not be created while the backend was
gone?

subtle, heh? So maybe we just need to mark doomed_ = true unconditionally in
MarkAsDoomed()?

[gavinp, 2013/09/12 15:39:48]

Done.

+    return net::OK;
   net_log_.AddEvent(net::NetLog::TYPE_SIMPLE_CACHE_ENTRY_DOOM_CALL);
   net_log_.AddEvent(net::NetLog::TYPE_SIMPLE_CACHE_ENTRY_DOOM_BEGIN);