Consider allowing insertTree to affect indexes

NEW
Unassigned

Status

()

Toolkit
Places
P3
normal
11 months ago
9 months ago

People

(Reporter: standard8, Unassigned)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(firefox57 affected)

Details

(Reporter)

Description

11 months ago
In bug 1385733 I'm trying to use insertTree in place of insert for inserting a folder.

browser_bookmarkProperties_addFolderDefaultButton.js was failing as the folder was getting inserted in the wrong position, even though the index was being passed to insertTree.

For now, PT.NewFolder includes an extra update after insertion to set the correct index.

Comment 1

9 months ago
The risk is that a consumer passes bogus indices, that's also the reason it doesn't pick indices iirc.

Updated

9 months ago
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.