Open Bug 1414435 Opened 2 years ago Updated Last year
Sort` from the bookmarks engine
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.
Hi @Kit. I'd like to be assigned to solve this bug. :) what exactly I need to do in this? replacement of 'defaultSort' with 'index'?
Hi Manish, thanks for your interest! This isn't quite ready to be worked on yet, because the old bookmarks engine still uses `_defaultSort: "index"`. We can remove `_defaultSort` after bug 1449730.
You need to log in before you can comment on or make changes to this bug.