Remove `_defaultSort` from the bookmarks engine

NEW
Unassigned

Status

()

Firefox
Sync
P3
normal
16 days ago
13 days ago

People

(Reporter: kitcambridge, Unassigned)

Tracking

(Depends on: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

We only set `_defaultSort` to "index" in the bookmarks engine (https://searchfox.org/mozilla-central/rev/af86a58b157fbed26b0e86fcd81f1b421e80e60a/services/sync/modules/engines/bookmarks.js#287), to fetch and apply toolbar bookmarks, folders, and frecent sites first.

Once bug 1305563 lands, the order of bookmark records won't matter, since we'll buffer everything before applying. Ryan also noted in bug 1368951, comment 4 that the Python server doesn't index on `sortindex`, so we might get a small efficiency boost from removing this and letting the server use whatever order it wants.
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.