Bug 1433807 - Removed unused `mirror.locations` index. r?tcsc draft
authorKit Cambridge <kit@yakshaving.ninja>
Mon, 29 Jan 2018 13:18:37 -0800
changeset 748461 604d55729173515a8c126ef07888e24277d77793
parent 748403 117e0c0d1ebe2cf5bdffc3474744add2416fc511
push id97171
push userbmo:kit@mozilla.com
push dateMon, 29 Jan 2018 21:20:52 +0000
reviewerstcsc
bugs1433807
milestone60.0a1
Bug 1433807 - Removed unused `mirror.locations` index. r?tcsc Since the synced bookmark mirror is currently disabled by default, we can do this without a migration. MozReview-Commit-ID: LWOwWCFHpuK
toolkit/components/places/SyncedBookmarksMirror.jsm
--- a/toolkit/components/places/SyncedBookmarksMirror.jsm
+++ b/toolkit/components/places/SyncedBookmarksMirror.jsm
@@ -1793,21 +1793,16 @@ async function initializeMirrorDatabase(
   await db.execute(`CREATE TABLE mirror.tags(
     itemId INTEGER NOT NULL REFERENCES items(id)
                             ON DELETE CASCADE,
     tag TEXT NOT NULL
   )`);
 
   await db.execute(`CREATE INDEX mirror.urlHashes ON urls(hash)`);
 
-  await db.execute(`CREATE INDEX mirror.locations ON structure(
-    parentGuid,
-    position
-  )`);
-
   await createMirrorRoots(db);
 }
 
 /**
  * Sets up the syncable roots. All items in the mirror should descend from these
  * roots. If we ever add new syncable roots to Places, this function should also
  * be updated to create them in the mirror.
  *