1074908 - Probably bad assertion in AssertMetadataConsistency()

Status: RESOLVED DUPLICATE of bug 1085575

(Core :: Storage: IndexedDB, defect)

Description

[Jan Varga [:janv]]

I added a test for the newly added file handle pref. The test sometimes fails (crashes the app) on Android 4.0 Debug.

I simplified the test (to not use file handle at all) and sent to try:
https://tbpl.mozilla.org/?tree=Try&rev=b40d9deb47b5

Here's the assertion:
Assertion failure: aThisObjectStore->mComittedAutoIncrementId <= otherObjectStore->mComittedAutoIncrementId

Comment 1

[Jan Varga [:janv]]

Attachment: failing test

Comment 2

[Jan Varga [:janv]]

The assertion is hit when the parent sends results for the second indexedDB.open()

Comment 3

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Hrm, I think this is probably a race. Can you reproduce reliably?

Comment 4

[Jan Varga [:janv]]

Yeah, looks like a race, here are some other try pushes:
https://tbpl.mozilla.org/?tree=Try&rev=3253a052d640

This one is almost all orange:
https://tbpl.mozilla.org/?tree=Try&rev=b3f8d80724f3

Unfortunately I can't reproduce locally, only on Android, on try.

Note, that there are no other hidden patches pushed to try, just the simple one with the mochitest.