Delete empty dom storage databases on destruction.

webkit/dom_storage/dom_storage_database_unittest.cc

Index: webkit/dom_storage/dom_storage_database_unittest.cc
diff --git a/webkit/dom_storage/dom_storage_database_unittest.cc b/webkit/dom_storage/dom_storage_database_unittest.cc
index 3f2ec2630dcd54abbe7d4e6581158eaa3772ca12..8ad4c9df745118d80b829dafd9060821dbe18d5b 100644
--- a/webkit/dom_storage/dom_storage_database_unittest.cc
+++ b/webkit/dom_storage/dom_storage_database_unittest.cc
@@ -113,6 +113,28 @@ TEST(DomStorageDatabaseTest, SimpleOpenAndClose) {
   EXPECT_FALSE(db.IsOpen());
 }
 
+TEST(DomStorageDatabaseTest, CloseEmptyDatabaseDeletesFile) {
+  ScopedTempDir temp_dir;
+  ASSERT_TRUE(temp_dir.CreateUniqueTempDir());
+  FilePath file_name = temp_dir.path().AppendASCII("TestDomStorageDatabase.db");
+  ValuesMap storage;
+  CreateMapWithValues(&storage);
+
+  {
+    DomStorageDatabase db(file_name);
+    ASSERT_TRUE(db.CommitChanges(false, storage));
+  }
+
+  EXPECT_TRUE(file_util::PathExists(file_name));
+  storage.clear();
+
+  {
+    DomStorageDatabase db(file_name);
+    ASSERT_TRUE(db.CommitChanges(true, storage));
+  }
+  EXPECT_FALSE(file_util::PathExists(file_name));
+}
+
 TEST(DomStorageDatabaseTest, TestLazyOpenIsLazy) {
   // This test needs to operate with a file on disk to ensure that we will
   // open a file that already exists when only invoking ReadAllValues.