Test for destroying leveldb directory when we detect a schema downgrade

Test for destroying leveldb directory when we detect corruption or a schema downgrade

BUG=156490, 158176
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=166600

[dgrogan, 2012-10-17 22:57:59]

Josh, could you review this?

[jsbell, 2012-10-17 23:44:14]

lgtm with really trivial nits.

There's probably more opportunity for refactoring when we add more tests that
depend on existing leveldb snapshots, but this looks good for now since we don't
know what will be common (i.e. will each test only do one CopyLevelDBToProfile
so it could be a method, or will there be more elaborate variants? will testing
the size be a common operation or not?)

https://codereview.chromium.org/11196029/diff/8002/content/test/data/indexedd...
File content/test/data/indexeddb/invalid_schema.js (right):

https://codereview.chromium.org/11196029/diff/8002/content/test/data/indexedd...
content/test/data/indexeddb/invalid_schema.js:10: request.onupgradeneeded =
upgradeNeeded;
Nit: maybe put onupgradeneeded before onsuccess to match event order?

https://codereview.chromium.org/11196029/diff/8002/content/test/data/indexedd...
content/test/data/indexeddb/invalid_schema.js:18: 
Nit: remove one blank line

[commit-bot: I haz the power, 2012-10-18 14:36:10]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dgrogan@chromium.org/11196029/6002

[commit-bot: I haz the power, 2012-10-18 14:36:14]

Can't process patch for file
content/test/data/indexeddb/schema_version_123456/file__0.indexeddb.leveldb/MANIFEST-000002.
Binary file support is temporarilly disabled due to a bug. Please commit blindly
the binary files first then commit the source change as a separate CL. Sorry for
the annoyance.

[jsbell, 2012-10-29 19:16:34]

It looks like this hasn't landed yet?

Please include an additional test for the case where the database is corrupt so
open fails. See http://webkit.org/b/100692

[dgrogan, 2012-11-06 23:00:40]

On 2012/10/29 19:16:34, jsbell wrote:
> It looks like this hasn't landed yet?
> 
> Please include an additional test for the case where the database is corrupt
so
> open fails. See http://webkit.org/b/100692

Done.  I took schema_version_123456/file__0.indexeddb.leveldb/000003.log,
overwrote the first 13 bytes with ascii 'A' and deleted some bytes out of the
middle.  That's good enough for a

ERROR: Failed to open LevelDB database from
/tmp/.org.chromium.Chromium.OdSmsA/IndexedDB/file__0.indexeddb.leveldb:
Corruption: bad record length


The CQ doesn't handle binary files.  I'm going to move the leveldb directories
to a separate CL, land it manually, then commit the remaining code changes
through the CQ.

[jsbell, 2012-11-07 19:53:12]

lgtm

[commit-bot: I haz the power, 2012-11-08 00:05:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dgrogan@chromium.org/11196029/22001

[commit-bot: I haz the power, 2012-11-08 00:36:18]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_rel.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_rel&...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2012-11-08 00:36:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dgrogan@chromium.org/11196029/19014

[commit-bot: I haz the power, 2012-11-08 04:38:47]

Change committed as 166600