wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 1244038 - Drop localStorage database indexes before renaming the table during update. r=mak77

Browse Source
This commit is contained in:
Honza Bambas 2016-02-01 12:32:00 -05:00
parent f9a2835cac
commit 5d5abc47a4
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
dom/storage/DOMStorageDBUpdater.cpp
View File

@ -336,7 +336,17 @@ nsresult Update(mozIStorageConnection *aWorkerConnection)
aWorkerConnection->RemoveFunction(NS_LITERAL_CSTRING("REVERSESTRING"));
// Update the scoping to match the new implememntation: split to oa suffix and origin key
// First rename the old table, we want to remove some columns no longer needed.
// First rename the old table, we want to remove some columns no longer needed,
// but even before that drop all indexes from it (CREATE IF NOT EXISTS for index on the
// new table would falsely find the index!)
rv = aWorkerConnection->ExecuteSimpleSQL(NS_LITERAL_CSTRING(
"DROP INDEX IF EXISTS webappsstore2.origin_key_index"));
NS_ENSURE_SUCCESS(rv, rv);
rv = aWorkerConnection->ExecuteSimpleSQL(NS_LITERAL_CSTRING(
"DROP INDEX IF EXISTS webappsstore2.scope_key_index"));
NS_ENSURE_SUCCESS(rv, rv);
rv = aWorkerConnection->ExecuteSimpleSQL(NS_LITERAL_CSTRING(
"ALTER TABLE webappsstore2 RENAME TO webappsstore2_old"));
NS_ENSURE_SUCCESS(rv, rv);