FileAPI: Run database recovery on IOError.

webkit/fileapi/file_system_directory_database_unittest.cc

Index: webkit/fileapi/file_system_directory_database_unittest.cc
diff --git a/webkit/fileapi/file_system_directory_database_unittest.cc b/webkit/fileapi/file_system_directory_database_unittest.cc
index 69eef26d34c14c5ee606d44378fef5b4bd6623ce..cfce4e99ebba15e45843711f3a98b59dea54f4db 100644
--- a/webkit/fileapi/file_system_directory_database_unittest.cc
+++ b/webkit/fileapi/file_system_directory_database_unittest.cc
@@ -650,4 +650,27 @@ TEST_F(FileSystemDirectoryDatabaseTest, TestRepairDatabase_Failure) {
   EXPECT_TRUE(db()->IsFileSystemConsistent());
 }
 
+TEST_F(FileSystemDirectoryDatabaseTest, TestRepairDatabase_MissingManifest) {
+  base::FilePath::StringType kFileName = FPL("bar");
+
+  FileId file_id_prev;
+  CreateFile(0, FPL("foo"), FPL("hoge"), NULL);
+  CreateFile(0, kFileName, FPL("fuga"), &file_id_prev);
+
+  const base::FilePath kDatabaseDirectory =
+      path().Append(kDirectoryDatabaseName);
+  CloseDatabase();
+
+  DeleteDatabaseFile(kDatabaseDirectory, leveldb::kDescriptorFile);

[kinuko, 2013/04/10 06:23:49]

What happens when other files are deleted?

[tzik, 2013/04/10 06:49:20]

For .sst deletion, that was handled as corruption.
Other file deletion for LOG, LOG.old, CURRENT, LOCK, lost/*  or *.log caused no
error.
MANIFEST-* file deletion seems only case that we didn't handled.

[kinuko, 2013/04/10 08:41:17]

Should we add tests for them too (even if they may not be handled by the new
code)?

[tzik, 2013/04/10 10:59:49]

Added tests for them on OriginDatabase test.
Should we also add for DirectoryDatabase?

[kinuko, 2013/04/10 11:02:13]

If you think we should do so please add them; I'm fine with this (if anything
changes in leveldb the other test will catch).

[tzik, 2013/04/10 13:01:09]

I believe it's enough to catch leveldb change. ;)

+
+  InitDatabase();
+  EXPECT_FALSE(db()->IsFileSystemConsistent());
+
+  FileId file_id;
+  EXPECT_TRUE(db()->GetChildWithName(0, kFileName, &file_id));
+  EXPECT_EQ(file_id_prev, file_id);
+
+  EXPECT_TRUE(db()->IsFileSystemConsistent());
+}
+
 }  // namespace fileapi