Bug 1700402 Comment 2 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 
It seems we have 2 pending requests on the transaction [when we get into ActorDestroy](https://searchfox.org/mozilla-central/rev/6c9ff2820d3aae683ec87c53c79e5108d54f3f76/dom/indexedDB/ActorsChild.cpp#2554). But we call `mTransaction->OnRequestFinished` just once. We should call it until [`mPendingRequestCount`](https://searchfox.org/mozilla-central/rev/6c9ff2820d3aae683ec87c53c79e5108d54f3f76/dom/indexedDB/IDBTransaction.cpp#349) is 0 (or provide a different OnRequestAbort function that does this) to force the commit.
Revision 1 by
on 
It seems we have 2 pending requests on the transaction [when we get into ActorDestroy](https://searchfox.org/mozilla-central/rev/6c9ff2820d3aae683ec87c53c79e5108d54f3f76/dom/indexedDB/ActorsChild.cpp#2554). But we call `mTransaction->OnRequestFinished` just once. We should call it until [`mPendingRequestCount`](https://searchfox.org/mozilla-central/rev/6c9ff2820d3aae683ec87c53c79e5108d54f3f76/dom/indexedDB/IDBTransaction.cpp#349) is 0 (or provide a different `OnRequestAbort` function that does this) to force the commit.
Revision 2 by
on 
It seems we have 2 pending requests on the transaction [when we get into ActorDestroy](https://searchfox.org/mozilla-central/rev/6c9ff2820d3aae683ec87c53c79e5108d54f3f76/dom/indexedDB/ActorsChild.cpp#2554). But we call `mTransaction->OnRequestFinished` just once. We should call it until [`mPendingRequestCount`](https://searchfox.org/mozilla-central/rev/6c9ff2820d3aae683ec87c53c79e5108d54f3f76/dom/indexedDB/IDBTransaction.cpp#349) is 0 (or provide a different `OnRequestAbort` function that does this) to force the commit?

Back to Bug 1700402 Comment 2