Not for review: Support sync init with missing or corrupt store

sync/syncable/directory_backing_store_unittest.cc

Index: sync/syncable/directory_backing_store_unittest.cc
diff --git a/sync/syncable/directory_backing_store_unittest.cc b/sync/syncable/directory_backing_store_unittest.cc
index 5161817f532de4be17f784851613aae5bfc08725..65dd1e8fbd99023accd570ba87fa7e5ae33a7ecb 100644
--- a/sync/syncable/directory_backing_store_unittest.cc
+++ b/sync/syncable/directory_backing_store_unittest.cc
@@ -2084,25 +2084,6 @@ TEST_F(DirectoryBackingStoreTest, ModelTypeIds) {
   }
 }
 
-// TODO(109668): This had to be disabled because the latest API will
-// intentionally crash when a database is this badly corrupted.
-TEST_F(DirectoryBackingStoreTest, DISABLED_Corruption) {

[rlarocque, 2012/06/04 20:07:09]

I've given up on testing this.  I simply can't find a way to trigger the right
kind of failure.  All my attempts are either unable to trigger an error, or
cause a CHECK failure in the SQL code.

-  {
-    scoped_ptr<OnDiskDirectoryBackingStore> dbs(
-        new OnDiskDirectoryBackingStore(GetUsername(), GetDatabasePath()));
-    EXPECT_TRUE(LoadAndIgnoreReturnedData(dbs.get()));
-  }
-  std::string bad_data("BAD DATA");
-  EXPECT_TRUE(file_util::WriteFile(GetDatabasePath(), bad_data.data(),
-                                   bad_data.size()));
-  {
-    scoped_ptr<OnDiskDirectoryBackingStore> dbs(
-        new OnDiskDirectoryBackingStore(GetUsername(), GetDatabasePath()));
-
-    EXPECT_FALSE(LoadAndIgnoreReturnedData(dbs.get()));
-  }
-}
-
 TEST_F(DirectoryBackingStoreTest, DeleteEntries) {
   sql::Connection connection;
   ASSERT_TRUE(connection.OpenInMemory());