New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SQLiteStorageArea should start new transaction when the previous one is rolled back by sqlite #9660
Conversation
EWS run on previous version of this PR (hash 0e6afe6) |
Typo: implciti |
0e6afe6
to
90e6d10
Compare
EWS run on current version of this PR (hash 90e6d10) |
β¦is rolled back by sqlite https://bugs.webkit.org/show_bug.cgi?id=251764 rdar://105061923 Reviewed by Chris Dumez. When debugging storage_local_setitem_quotaexceedederr.window.html failure, I found SQLiteStorageArea does not start new transaction after SQLITE_FULL error, which means it would create an implicit transaction for each statement. To fix this, this patch makes startTransactionIfNecessary() check whether existing transaction is already rolled back. If the transaction is rolled back, it should start a new transaction. * Source/WebCore/platform/sql/SQLiteTransaction.h: * Source/WebKit/NetworkProcess/storage/SQLiteStorageArea.cpp: (WebKit::SQLiteStorageArea::startTransactionIfNecessary): Canonical link: https://commits.webkit.org/259966@main
90e6d10
to
7833f1f
Compare
Committed 259966@main (7833f1f): https://commits.webkit.org/259966@main Reviewed commits have been landed. Closing PR #9660 and removing active labels. |
7833f1f
90e6d10
π iosπ macπ wincairoπ§ͺ ios-wk2π§ͺ api-macπ§ͺ gtk-wk2π§ͺ api-iosπ§ͺ mac-wk1π§ͺ api-gtkπ tvπ§ͺ mac-wk2π tv-simπ§ͺ mac-AS-debug-wk2π watchπ§ͺ mac-wk2-stressπ watch-sim