Closed Bug 484096 Opened 15 years ago Closed 10 years ago

nsNavBookmarks::SetItemIndex is really inefficient about its listeners

Categories

(Toolkit :: Places, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX

People

(Reporter: sdwilsh, Unassigned)

References

Details

It dispatches an item removed, and then an item added when only the index changes.  Seems to me like it should really be dispatching OnItemChagned with the bookmark id, property = "index", and the old index (they can query the item to get the new index).
That, or make a new listener topic for this
Even better - onItemMoved
Depends on: 539517
we will remove this API
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.