Consider changing bookmarks.update to apply transactions to multiple bookmarks to aid performance

NEW
Unassigned

Status

()

enhancement
P3
normal
2 years ago
Last year

People

(Reporter: standard8, Unassigned)

Tracking

({perf})

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox59 affected)

Details

(Whiteboard: [fxsearch])

Reporter

Description

2 years ago
In bug 1404909 we're changing bookmarks.update to handle multiple items to update at the same time.

One improvement that we might want to experiment with is to change the executeTransaction wrapper, so that it wraps multiple items, rather than each individual item. See bug 1404909 comment 8.

To do this, we might need to handle chunking as well.

Also, it might be worth at the same time, considering if there is a way to simplify the setAncestorsLastModified calls, as per bug 1404909 comment 9.
Priority: -- → P3
Whiteboard: [fxsearch]
You need to log in before you can comment on or make changes to this bug.