Bug 1986816 Comment 3 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
GetStatementOnHelperThread should probably check mIsConnectionReady before trying to use mHelperThreadStatements.
GetCachedStatement should probably hold sharedAsyncExecutionMutex, because it looks like the connection is being teared down.
Revision 1 by
on 
GetStatementOnHelperThread should probably check mIsConnectionReady before trying to use mHelperThreadStatements.
~~GetCachedStatement should probably hold sharedAsyncExecutionMutex, because it looks like the connection is being teared down.~~ <= this is not necessary, StatementCache doesn't own the connection and relies on the consumer to properly handle teardown.

Back to Bug 1986816 Comment 3