Do not do unnecessary recursive deletes for files.

components/data_reduction_proxy/core/browser/data_store_impl.cc

Index: components/data_reduction_proxy/core/browser/data_store_impl.cc
diff --git a/components/data_reduction_proxy/core/browser/data_store_impl.cc b/components/data_reduction_proxy/core/browser/data_store_impl.cc
index a34407e6ddb21585df239d8e739dac33b63d64ed..78530e0a37b592e7bc0dd3e74086d6e9747947d6 100644
--- a/components/data_reduction_proxy/core/browser/data_store_impl.cc
+++ b/components/data_reduction_proxy/core/browser/data_store_impl.cc
@@ -147,7 +147,7 @@ void DataStoreImpl::RecreateDB() {
 
   LOG(WARNING) << "Deleting corrupt Data Reduction Proxy LevelDB";
   db_.reset(nullptr);
-  base::DeleteFile(profile_path_.Append(kDBName), true);
+  base::DeleteFile(profile_path_.Append(kDBName), false);

[Not at Google. Contact bengr, 2015/11/30 19:50:24]

It seems to me that the recursive delete is necessary here.

Given that the LevelDB is corrupt, we should delete all the files. This is
expected to happen very rarely, so not sure there is any observable benefit to
making this non-recursive.

The same should be true of all cases in this cl which are deleting a corrupt
LevelDB.

[Lei Zhang, 2015/12/03 21:39:06]

Ah, I see, LevelDB takes a directory and not a file. Reverted.

 
   OpenDB();
 }